Opened 11 years ago

Closed 11 years ago

#8512 closed defect (fixed)

Optional database_stein_watkins_mini uses 'cp -v' which fails on Solaris.

Reported by: drkirkby Owned by: drkirkby
Priority: major Milestone: sage-4.4.4
Component: packages: optional Keywords:
Cc: was Merged in:
Authors: David Kirkby Reviewers: William Stein
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Description (last modified by drkirkby)

The optional package "database_stein_watkins_mini" fails to install on Solaris 10:

Hardware & associated software

  • Sun Blade 1000
  • 2 x 900 MHz UltraSPARC III+ CPUs
  • 2 GB RAM
  • Solaris 10 03/2005 (first release of Solaris 10)
  • gcc 4.4.3 (uses Sun linker and assembler)

Sage version

  • 4.3.4.alpha1
  • Patch #8509 removing the -o option to grep to allow packages to install.

The problem

database_stein_watkins_mini/spkg-install
Finished extraction
****************************************************
Host system
uname -a:
SunOS redstart 5.10 Generic sun4u sparc SUNW,Sun-Blade-1000
****************************************************
****************************************************
CC Version
gcc -v
Using built-in specs.
Target: sparc-sun-solaris2.10
Configured with: ../gcc-4.4.3/configure --prefix=/usr/local/gcc-4.4.3 --with-mpfr=/usr/local/gcc-4.4.3 --with-build-time-tools=/usr/ccs/bin --with-gmp=/usr/local/gcc-4.4.3 --enable-languages=c,c++,fortran
Thread model: posix
gcc version 4.4.3 (GCC)
****************************************************
cp: illegal option -- v
Usage: cp [-f] [-i] [-p] [-@] f1 f2
       cp [-f] [-i] [-p] [-@] f1 ... fn d1
       cp -r|-R [-H|-L|-P] [-f] [-i] [-p] [-@] d1 ... dn-1 dn

real    0m0.013s
user    0m0.004s
sys     0m0.009s
sage: An error occurred while installing database_stein_watkins_mini

The solution

This is because 'cp' uses an illegal option -v, which is not defined by POSIX.

http://www.opengroup.org/onlinepubs/9699919799/utilities/cp.html

Since the GNU version of 'cp' only uses the -v option to show what is being done, it is not important. Consulting rom the 'cp' man page on Linux:

       -v, --verbose
              explain what is being done

The -v option can simply be removed.

Attachments (2)

SPKG.txt (389 bytes) - added by drkirkby 11 years ago.
Brand new (but mostly incomplete) SPKG.txt
stein-watkins-ecdb.patch (251 bytes) - added by drkirkby 11 years ago.
Modified spkg-install, removing the '-v' option to 'cp'

Download all attachments as: .zip

Change History (8)

comment:1 Changed 11 years ago by drkirkby

  • Authors set to David Kirkby
  • Status changed from new to needs_review

The solution to this GNUism was very easy - just remove the -v option.

I also added a very incomplete SPKG.txt file - previously there was no file called SPKG.txt. I leave it up to William or others with more knowledge to fill this in. It does at least document the change I made.

The database can be found at http://boxen.math.washington.edu/home/kirkby/optional/database_stein_watkins_mini.p0/database_stein_watkins_mini.p0.spkg

Attached are a diff of the spkg-install and the new SPKG.txt

Note to the release manager.

There is no Mercurial repository - I this will have to be integrated manually.

Changed 11 years ago by drkirkby

Brand new (but mostly incomplete) SPKG.txt

Changed 11 years ago by drkirkby

Modified spkg-install, removing the '-v' option to 'cp'

comment:2 Changed 11 years ago by drkirkby

  • Description modified (diff)

comment:3 Changed 11 years ago by drkirkby

  • Summary changed from database_stein_watkins_mini uses 'cp -v' which fails on Solaris. to Optional database_stein_watkins_mini uses 'cp -v' which fails on Solaris.

comment:4 Changed 11 years ago by drkirkby

  • Description modified (diff)
  • Owner changed from tbd to drkirkby

comment:5 Changed 11 years ago by was

  • Status changed from needs_review to positive_review

comment:6 Changed 11 years ago by mhansen

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