Opened 8 years ago

Closed 6 months ago

#15136 closed enhancement (duplicate)

spkg-install improvements

Reported by: felixs Owned by:
Priority: major Milestone: sage-duplicate/invalid/wontfix
Component: packages: standard Keywords: spkg-install idempotence environment filelist
Cc: Merged in:
Authors: Reviewers: Dima Pasechnik
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: #14750, #14804 Stopgaps:

Status badges

Description (last modified by felixs)

The build system #14796 works better with some minor tweaks to some spkg-install scripts.

  • with idempotence, "keepbuilt" starts to make sense
  • without hardwired paths, compiling with packages disabled (=fallback to system) works
  • with environment in $SAGE_ETC/env.d/*, run time environment is more flexible
  • with a list of files that belong to a package, unstaging is easy.
  • setting write permissions for copied files allows cleanup, even if the source files are not writable

these changes are minimal (for each package) and are intended to not break the old build system. This is a series of individual patches. Opening up ~100 tickets doesn't seem appropriate.

Change History (15)

comment:1 Changed 8 years ago by felixs

  • Branch set to u/felixs/spkg-install
  • Description modified (diff)
  • Keywords filelist added

comment:2 Changed 8 years ago by git

  • Commit set to 056ae2b4eaae9f52dd867bbb79ad914a49768a74

Branch pushed to git repo; I updated commit sha1. New commits:

[changeset:056ae2b]gcc: spkg-install idempotence
[changeset:0147680]gap: spkg-install
[changeset:0b18022]flintqs: spkg-install
[changeset:69d0d54]flint: spkg-install
[changeset:8dde291]fflas_ffpack: spkg-install, fix idempotence
[changeset:ff10222]elliptic_curves: spkg-install, fix idempotence
[changeset:e15cbc7]ecm: spkg-install misc
[changeset:15b3405]eclib: spkg
[changeset:522420f]ecl: spkg-install
[changeset:e41996c]docutils: use sage-setup.py
[changeset:c7e949b]cython: spkg-install
[changeset:a2e0f49]cvxopt: spkg-install, idempotence, filelist
[changeset:d076fdb]conway_polynomials: spkg-install
[changeset:5c52c6e]cliquer: spkg-install
[changeset:135b66b]cephes: patch quickfix
[changeset:d1e078b]cddlib: fix permissions
[changeset:9b56d97]boehm_gc: spkg-install

comment:3 Changed 8 years ago by felixs

  • Description modified (diff)
  • Status changed from new to needs_review

I have set "needs_review", subsequent commits will be independent.

comment:4 Changed 8 years ago by git

  • Commit changed from 056ae2b4eaae9f52dd867bbb79ad914a49768a74 to 06d1ff120d45a17ca5d77ecc6dfde648eeebf61b

Branch pushed to git repo; I updated commit sha1. New commits:

[changeset:06d1ff1]mpir: spkg-install
[changeset:e047964]mpc: spkg-install idempotence, cleanup
[changeset:7565e7d]maxima: spkg-install
[changeset:68c1a8d]matplotlib: spkg-install
[changeset:1a0fe61]linbox: spkg-install
[changeset:786c1bc]lcalc: spkg-install idempotence
[changeset:d29271e]lapack: spkg-install
[changeset:08ca8c2]jinja2: spkg-install, filelist etc
[changeset:0419220]ipython spkg-install idempotence
[changeset:7163f7f]iml: spkg-install idempotence
[changeset:477b152]givaro: spkg-install
[changeset:e4b0ac8]graphs spkg-install
[changeset:de59617]gsl: spkg-install idempotence
[changeset:f956538]gfan: spkg-install
[changeset:939287a]gdmodule: spkg-install

comment:5 Changed 8 years ago by git

  • Commit changed from 06d1ff120d45a17ca5d77ecc6dfde648eeebf61b to ccf9e10ab151dc3830226a5ef168f5f9c5b61e69

Branch pushed to git repo; I updated commit sha1. New commits:

[changeset:ccf9e10]rpy2: spkg-install
[changeset:794dd8b]readline: spkg-install idempotence
[changeset:26f4f8a]ratpoints: spkg-install idempotence
[changeset:6767c87]r: spkg-install
[changeset:b8238a4]python: spkg-install
[changeset:b5d1421]pygments: spkg-install idempotence
[changeset:85cbe8d]pycrypto: filelist
[changeset:190ea4a]ppl: spkg-install idempotence
[changeset:1298667]polytopes_db: spkg-install
[changeset:c81193a]polybori: spkg-install
[changeset:f25da01]pil: spkg-install
[changeset:cfb562c]pexpect: spkg-install
[changeset:f8091b6]pari: spkg
[changeset:8c31502]palp: spkg-install
[changeset:0cae935]numpy: spkg-install
[changeset:bb1afa3]ntl: spkg-install idempotence
[changeset:270967d]ncurses: idempotence
[changeset:3842e72]mpmath: spkg-install, sage-setup.py, filelist

comment:6 Changed 8 years ago by git

  • Commit changed from ccf9e10ab151dc3830226a5ef168f5f9c5b61e69 to 77481b88f10227dd9f61743eff17da97b3a09594

Branch pushed to git repo; I updated commit sha1. New commits:

[changeset:77481b8]atlas: spkg-install
[changeset:3388eb3]zn_poly: idempotence
[changeset:3576102]zlib: spkg-install idempotence
[changeset:a4b7767]termcap: spkg-install idempotence
[changeset:17a1b82]sympow: spkg
[changeset:93cf3a3]symmetrica: spkg-install
[changeset:09055d0]sqlite: spkg-install idempotence
[changeset:fabcb32]sqlalchemy: filelist
[changeset:20c5c9f]sphinx: spkg-install
[changeset:5ded6fc]setuptools: spkg-install
[changeset:0190cf6]scipy spkg-install idempotence
[changeset:91e423d]sagetex: spkg-install, makefile, filelist

comment:7 Changed 8 years ago by git

  • Commit changed from 77481b88f10227dd9f61743eff17da97b3a09594 to f623de803e6818d8101403018eb2108ff62292c1

Branch pushed to git repo; I updated commit sha1. New commits:

[changeset:f623de8]flint: create directory before putting file into it
[changeset:80c1cd3]pari: cleanup
[changeset:a51658f]tachyon. make sure "bin" exists, and use spkg-dist-install
[changeset:2ae993b]linbox: remove extra/broken CFLAGS env
[changeset:c166678]delete stray backup

comment:8 Changed 7 years ago by jdemeyer

  • Status changed from needs_review to needs_work

Needs to be rebased.

comment:9 Changed 7 years ago by vbraun_spam

  • Milestone changed from sage-6.1 to sage-6.2

comment:10 Changed 7 years ago by vbraun_spam

  • Milestone changed from sage-6.2 to sage-6.3

comment:11 Changed 7 years ago by vbraun_spam

  • Milestone changed from sage-6.3 to sage-6.4

comment:12 Changed 4 years ago by embray

It's too bad this never went anywhere since it was a lot of work. It seems to be redundant though with what I'm doing with #23160 #22509, and #22510.

comment:13 Changed 12 months ago by mkoeppe

  • Milestone changed from sage-6.4 to sage-duplicate/invalid/wontfix
  • Status changed from needs_work to needs_review

Outdated, should be closed

comment:14 Changed 6 months ago by dimpase

  • Reviewers set to Dima Pasechnik
  • Status changed from needs_review to positive_review

comment:15 Changed 6 months ago by slelievre

  • Authors Felix Salfelder deleted
  • Branch u/felixs/spkg-install deleted
  • Commit f623de803e6818d8101403018eb2108ff62292c1 deleted
  • Resolution set to duplicate
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.