Opened 7 years ago

Closed 7 years ago

#15766 closed enhancement (fixed)

sage-spkg can show harmless "mv" error

Reported by: jdemeyer Owned by:
Priority: major Milestone: sage-6.2
Component: build Keywords:
Cc: vbraun, ohanar Merged in:
Authors: Jeroen Demeyer Reviewers: R. Andrew Ohana
Report Upstream: N/A Work issues:
Branch: u/jdemeyer/ticket/15766 (Commits) Commit: c83506a06186c32523b077980b55a350399f99aa
Dependencies: Stopgaps:

Description (last modified by jdemeyer)

The mv command in

if [ "$USE_LOCAL_SCRIPTS" = yes ]; then
    mv "${PKG_NAME_UPSTREAM%.tar*}" src
    cd ..
    echo "Finished set up"
else

can sometimes give an error which is harmless. So we should silence this error. We can also remove some extra cd commands.

Attachments (1)

15766.patch (1.3 KB) - added by jdemeyer 7 years ago.

Download all attachments as: .zip

Change History (8)

Changed 7 years ago by jdemeyer

comment:1 Changed 7 years ago by vbraun_spam

  • Milestone changed from sage-6.1 to sage-6.2

comment:2 Changed 7 years ago by jdemeyer

  • Branch set to u/jdemeyer/ticket/15766
  • Created changed from 01/30/14 16:23:49 to 01/30/14 16:23:49
  • Modified changed from 01/30/14 21:20:52 to 01/30/14 21:20:52

comment:3 Changed 7 years ago by git

  • Commit set to c83506a06186c32523b077980b55a350399f99aa

Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:

c83506aSilence "mv" and eliminate some "cd" commands

comment:4 Changed 7 years ago by jdemeyer

  • Authors set to Jeroen Demeyer
  • Status changed from new to needs_review

comment:5 Changed 7 years ago by jdemeyer

  • Cc vbraun ohanar added
  • Description modified (diff)

comment:6 Changed 7 years ago by ohanar

  • Reviewers set to R. Andrew Ohana
  • Status changed from needs_review to positive_review

comment:7 Changed 7 years ago by vbraun

  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.