Opened 4 years ago

Closed 4 years ago

#18126 closed enhancement (fixed)

Coin-OR CBC: new-style SPKG

Reported by: ncohen Owned by:
Priority: major Milestone: sage-6.6
Component: packages: optional Keywords: sd66
Cc: tmonteil, vdelecroix, dcoudert Merged in:
Authors: Nathann Cohen Reviewers: David Coudert, Thierry Monteil, Vincent Delecroix
Report Upstream: N/A Work issues:
Branch: a328985 (Commits) Commit: a3289854b1d4da2d1b126ecc788f17966482df04
Dependencies: Stopgaps:

Description (last modified by vdelecroix)

This branch turns CBC into a new-style spkg.

SPKG: http://www.coin-or.org/download/source/Cbc/Cbc-2.9.4.tgz (has to be renamed cbc-2.9.4.tar.gz and stored in SAGE_ROOT/upstream)

Nathann

Change History (62)

comment:1 Changed 4 years ago by ncohen

  • Branch set to public/18126
  • Commit set to 97dd3652348c13847cfedc7e08d1fc05a8266ea4
  • Status changed from new to needs_review

New commits:

9e72fa1trac #18126: copy of spkg-install and SPKG.txt from the old style spkg
138570etrac #18126: new checksum and package-version files
97dd365trac #18126: new spkg-src file to download and build the spkg

comment:2 Changed 4 years ago by ncohen

  • Keywords sd66 added

comment:3 follow-up: Changed 4 years ago by dcoudert

  • Status changed from needs_review to needs_work

Are you sure it's IBM Coportation ?

I have the following issue, and I don't find it at that address.

confetti:sage dcoudert$ ./sage -i cbc
Found local metadata for cbc-2.9.4
Attempting to download package cbc-2.9.4
>>> Trying to download http://www.sagemath.org/packages/upstream/cbc/cbc-2.9.4.tar.bz2
[Traceback (most recent call last):
  File "<stdin>", line 35, in <module>
  File "/Users/dcoudert/sage/local/lib/python/urllib.py", line 240, in retrieve
    fp = self.open(url, data)
  File "/Users/dcoudert/sage/local/lib/python/urllib.py", line 208, in open
    return getattr(self, name)(url)
  File "/Users/dcoudert/sage/local/lib/python/urllib.py", line 359, in open_http
    return self.http_error(url, fp, errcode, errmsg, headers)
  File "/Users/dcoudert/sage/local/lib/python/urllib.py", line 376, in http_error
    return self.http_error_default(url, fp, errcode, errmsg, headers)
  File "<stdin>", line 17, in http_error_default
IOError: [Errno 404] Not Found: '//www.sagemath.org/packages/upstream/cbc/cbc-2.9.4.tar.bz2'
Error: failed to download package cbc-2.9.4

comment:4 Changed 4 years ago by git

  • Commit changed from 97dd3652348c13847cfedc7e08d1fc05a8266ea4 to 3bb47c9b70a20bbca5a835f2db47a5f592421373

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

3bb47c9trac #18126: License update

comment:5 in reply to: ↑ 3 Changed 4 years ago by ncohen

Hellooooo,

Are you sure it's IBM Coportation ?

While updating that I noticed that the license had changed. Fixed.

I have the following issue, and I don't find it at that address.

This is because you do not have the new .tar.bz2 file. The spkg-src script (see the ticket's description) that this branch contains will download it for you.

Nathann

comment:6 Changed 4 years ago by ncohen

  • Status changed from needs_work to needs_review

comment:7 follow-up: Changed 4 years ago by dcoudert

Hello,

still not working. Also, I don't see a cbc page at http://www.sagemath.org/packages/upstream/

Why not pointing directly to http://www.coin-or.org/download/source/Cbc/Cbc-2.9.4.tgz ?

David.

comment:8 in reply to: ↑ 7 Changed 4 years ago by ncohen

Hello,

still not working.

Could you give a more informative error message? What happens when you run spkg-src?

Also, I don't see a cbc page at http://www.sagemath.org/packages/upstream/

No problem here, that's because coin is an optional package. You don't see Nauty there either.

Why not pointing directly to http://www.coin-or.org/download/source/Cbc/Cbc-2.9.4.tgz ?

Because it is not a .tar.bz2 file, and that's what I think we have to put in our upstream/ folder.

Nathann

comment:9 follow-up: Changed 4 years ago by dcoudert

Hello,

I should run spkg-src and not sage -i cbc ??? You should mention it in the ticket description.

confetti:sage dcoudert$ build/pkgs/cbc/spkg-src 
cat: package-version.txt: No such file or directory
sed: illegal option -- r
usage: sed script [-Ealn] [-i extension] [file ...]
       sed [-Ealn] [-i extension] [-e script] ... [-f script_file] ... [file ...]
--2015-04-06 14:08:36--  http://www.coin-or.org/download/source/Cbc/Cbc-.tgz
Résolution de www.coin-or.org (www.coin-or.org)… 130.127.215.8
Connexion à www.coin-or.org (www.coin-or.org)|130.127.215.8|:80… connecté.
requête HTTP transmise, en attente de la réponse… 404 Not Found
2015-04-06 14:08:39 erreur 404 : Not Found.

confetti:sage dcoudert$ ./sage -i cbc
Found local metadata for cbc-2.9.4
Attempting to download package cbc-2.9.4
>>> Trying to download http://www.sagemath.org/packages/upstream/cbc/cbc-2.9.4.tar.bz2
[Traceback (most recent call last):
  File "<stdin>", line 35, in <module>
  File "/Users/dcoudert/sage/local/lib/python/urllib.py", line 240, in retrieve
    fp = self.open(url, data)
  File "/Users/dcoudert/sage/local/lib/python/urllib.py", line 208, in open
    return getattr(self, name)(url)
  File "/Users/dcoudert/sage/local/lib/python/urllib.py", line 359, in open_http
    return self.http_error(url, fp, errcode, errmsg, headers)
  File "/Users/dcoudert/sage/local/lib/python/urllib.py", line 376, in http_error
    return self.http_error_default(url, fp, errcode, errmsg, headers)
  File "<stdin>", line 17, in http_error_default
IOError: [Errno 404] Not Found: '//www.sagemath.org/packages/upstream/cbc/cbc-2.9.4.tar.bz2'
Error: failed to download package cbc-2.9.4

comment:10 in reply to: ↑ 9 Changed 4 years ago by ncohen

Hello,

I should run spkg-src and not sage -i cbc ???

spkg-src downloads the upstream Cbc for you, transforms it a bit and stores it in upstream. Could you start this script while going in the right folder first?

The spkg-src scripts are just there to make the developer's life easier when a spkg gets udpated. You change the version number, start the script, and the .tar.bz2 file is built for you.

Once this is done, the command sage -i cbc will find the .tar.bz2 file.

Nathann

comment:11 Changed 4 years ago by ncohen

Additionally, these spkg-src scripts are also useful to certify that the guy who uploads a .tar.bz2 file has not introduced any code in the original package. As you can see how the .tar.bz2 file is built, I cannot be suspected of introducing backdoors on everybody's computer.

Nathann

comment:12 Changed 4 years ago by dcoudert

confetti:cbc dcoudert$ ./spkg-src 
sed: illegal option -- r
usage: sed script [-Ealn] [-i extension] [file ...]
       sed [-Ealn] [-i extension] [-e script] ... [-f script_file] ... [file ...]
--2015-04-06 14:35:26--  http://www.coin-or.org/download/source/Cbc/Cbc-.tgz
Résolution de www.coin-or.org (www.coin-or.org)… 130.127.215.8
Connexion à www.coin-or.org (www.coin-or.org)|130.127.215.8|:80… connecté.
requête HTTP transmise, en attente de la réponse… 404 Not Found
2015-04-06 14:35:27 erreur 404 : Not Found.
confetti:cbc dcoudert$ cat package-version.txt | sed -re "s/^(.*)\\.p.+$/\\1/"
sed: illegal option -- r
usage: sed script [-Ealn] [-i extension] [file ...]
       sed [-Ealn] [-i extension] [-e script] ... [-f script_file] ... [file ...]
confetti:cbc dcoudert$ cat package-version.txt | sed -e "s/^(.*)\\.p.+$/\\1/"
sed: 1: "s/^(.*)\.p.+$/\1/
": \1 not defined in the RE
confetti:cbc dcoudert$ cat package-version.txt | sed -e "s/^(.*)\\.p.+$/$1/"
2.9.4

comment:13 Changed 4 years ago by git

  • Commit changed from 3bb47c9b70a20bbca5a835f2db47a5f592421373 to 5c7e2098647a54eab41d1130257bf0874762d93e

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

5c7e209trac #18126: avoid -r in sed

comment:14 follow-up: Changed 4 years ago by dcoudert

Some progress

confetti:sage dcoudert$ ./sage -i cbc
Found local metadata for cbc-2.9.4
Attempting to download package cbc-2.9.4
>>> Trying to download http://www.sagemath.org/packages/upstream/cbc/cbc-2.9.4.tar.bz2
[Traceback (most recent call last):
  File "<stdin>", line 35, in <module>
  File "/Users/dcoudert/sage/local/lib/python/urllib.py", line 240, in retrieve
    fp = self.open(url, data)
  File "/Users/dcoudert/sage/local/lib/python/urllib.py", line 208, in open
    return getattr(self, name)(url)
  File "/Users/dcoudert/sage/local/lib/python/urllib.py", line 359, in open_http
    return self.http_error(url, fp, errcode, errmsg, headers)
  File "/Users/dcoudert/sage/local/lib/python/urllib.py", line 376, in http_error
    return self.http_error_default(url, fp, errcode, errmsg, headers)
  File "<stdin>", line 17, in http_error_default
IOError: [Errno 404] Not Found: '//www.sagemath.org/packages/upstream/cbc/cbc-2.9.4.tar.bz2'
Error: failed to download package cbc-2.9.4
confetti:sage dcoudert$          
confetti:sage dcoudert$ cd build/pkgs/cbc/
confetti:cbc dcoudert$ ./spkg-src 
--2015-04-06 14:48:46--  http://www.coin-or.org/download/source/Cbc/Cbc-2.9.4.tgz
Résolution de www.coin-or.org (www.coin-or.org)… 130.127.215.8
Connexion à www.coin-or.org (www.coin-or.org)|130.127.215.8|:80… connecté.
requête HTTP transmise, en attente de la réponse… 200 OK
Taille : 13761766 (13M) [application/x-gzip]
Sauvegarde en : « Cbc-2.9.4.tgz »

Cbc-2.9.4.tgz                                       100%[===================================================================================================================>]  13,12M  74,6KB/s   ds 2m 37s 

2015-04-06 14:51:24 (85,5 KB/s) — « Cbc-2.9.4.tgz » sauvegardé [13761766/13761766]

mv: rename Cbc-2.9.4.tar.bz2 to /upstream/cbc-2.9.4.tar.bz2: No such file or directory
confetti:cbc dcoudert$ ll
total 23032
-rw-r--r--  1 dcoudert  staff  11768072 31 mar 09:30 Cbc-2.9.4.tar.bz2
-rw-r--r--  1 dcoudert  staff       905  6 avr 14:04 SPKG.txt
-rw-r--r--  1 dcoudert  staff       128  6 avr 14:04 checksums.ini
-rw-r--r--  1 dcoudert  staff         6  6 avr 14:04 package-version.txt
-rwxr-xr-x  1 dcoudert  staff      1220  6 avr 14:04 spkg-install
-rwxr-xr-x  1 dcoudert  staff       429  6 avr 14:47 spkg-src

Also, version 2.9.4 is not listed on page http://www.sagemath.org/packages/optional/ Only 2.8.1.p0

comment:15 in reply to: ↑ 14 Changed 4 years ago by ncohen

Hello,

This fails because your SAGE_ROOT variable is not set. You can:

1) Define it in your bash; or 2) Run the script from the console that you get after a sage -sh

Nathann

comment:16 Changed 4 years ago by git

  • Commit changed from 5c7e2098647a54eab41d1130257bf0874762d93e to 217fb18a0d198ba2966fa3973fe845d9eb89a0e1

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

217fb18trac #18126: Warning when SAGE_ROOT is not set

comment:17 Changed 4 years ago by git

  • Commit changed from 217fb18a0d198ba2966fa3973fe845d9eb89a0e1 to 4e3830fbf8c6c7d72ba6f0cdf44ee761bc203a2a

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

4e3830ftrac #18126: Warning when SAGE_ROOT is not set

comment:18 follow-up: Changed 4 years ago by dcoudert

You did a push while I was testing the stuff, so I don't know if it will change something.

After setting SAGE_ROOT in .bash_profile :

confetti:cbc dcoudert$ ./spkg-src 
--2015-04-06 15:07:53--  http://www.coin-or.org/download/source/Cbc/Cbc-2.9.4.tgz
Résolution de www.coin-or.org (www.coin-or.org)… 130.127.215.8
Connexion à www.coin-or.org (www.coin-or.org)|130.127.215.8|:80… connecté.
requête HTTP transmise, en attente de la réponse… 200 OK
Taille : 13761766 (13M) [application/x-gzip]
Sauvegarde en : « Cbc-2.9.4.tgz »

Cbc-2.9.4.tgz                                       100%[===================================================================================================================>]  13,12M  93,9KB/s   ds 2m 9s  

2015-04-06 15:10:05 (104 KB/s) — « Cbc-2.9.4.tgz » sauvegardé [13761766/13761766]

./spkg-src: line 17: sage: command not found
confetti:cbc dcoudert$ echo $SAGE_ROOT
/Users/dcoudert/sage/
confetti:cbc dcoudert$ sage
┌────────────────────────────────────────────────────────────────────┐
│ SageMath Version 6.6.rc2, Release Date: 2015-04-02                 │
│ Type "notebook()" for the browser-based notebook interface.        │
│ Type "help()" for help.                                            │
└────────────────────────────────────────────────────────────────────┘
┏━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━┓
┃ Warning: this is a prerelease version, and it may be unstable.     ┃
┗━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━┛
sage: 
Exiting Sage (CPU time 0m0.02s, Wall time 0m5.00s).
confetti:cbc dcoudert$ cd $SAGE_ROOT
confetti:sage dcoudert$ sage -i cbc
Found local metadata for cbc-2.9.4
Found local sources at /Users/dcoudert/sage/upstream/cbc-2.9.4.tar.bz2
Checksum: 6afb943dcc3f60bff525456ad175cf19f6a2e28e vs 6afb943dcc3f60bff525456ad175cf19f6a2e28e
cbc-2.9.4
====================================================
Setting up build directory for cbc-2.9.4
...
...
Successfully installed cbc-2.9.4
Deleting temporary build directory
/Users/dcoudert/sage/local/var/tmp/sage/build/cbc-2.9.4
Finished installing cbc-2.9.4.spkg

Warning: it might be needed to update the Sage library before
installed packages work: you should run 'make' from $SAGE_ROOT
before running Sage.

So it is working this way, but it was not working when trying first sage -i cbc. Don't know why.

What should I do next?

comment:19 in reply to: ↑ 18 Changed 4 years ago by ncohen

Hello !

You did a push while I was testing the stuff, so I don't know if it will change something.

I added a warning to say something more useful when SAGE_ROOT is not defined. This block of 4 lines can be found in most spkg-install scripts that Sage ships.

So it is working this way, but it was not working when trying first sage -i cbc. Don't know why.

New-style spkg work this way: ideally we have in upstream/ the copy of the source files, as given by the original authors of the software. Then, in build/pkgs we have some scripts to install it as it should in Sage.

As the sources were not available on your computer (and obviously not on Sage's server either, as the version we have here is more recent) the spkg-src script downloaded it from Coin's website and created the upstream/ archive. Then, with sage -i cbc you called the scripts from build/pkgs/, which used the new .tar.bz2 file.

What should I do next?

Well, now you apparently have Cbc on your computer. So you can solve LP with it, check that tests pass when you add --optional=sage,coin to sage -t.

Nathann

comment:20 follow-up: Changed 4 years ago by tmonteil

comment:21 Changed 4 years ago by tmonteil

  • Status changed from needs_review to needs_work

comment:22 in reply to: ↑ 20 ; follow-up: Changed 4 years ago by ncohen

Hello,

  • The changes made on the source code should better be in a separate patch file rather than a sed command within the spkg-install script (so that one can easily see what changed from upstream in a uniform way). The associated comment should be part of the patch, see #17357

The spkg-install file is a pure copy of what the spkg already contains.

Because I believe that we need .tar.bz2 files in upstream. There is no other reason.

I don't mind. I will remove it in a second and set the patch to needs_review again, waiting for your answer about the first two points.

Nathann

comment:23 Changed 4 years ago by git

  • Commit changed from 4e3830fbf8c6c7d72ba6f0cdf44ee761bc203a2a to a372011eb60f2a3e6b01831430743a7421fd56f8

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

a372011trac #18126: Remove the 'maintainers' section from SPKG.txt

comment:24 Changed 4 years ago by ncohen

  • Status changed from needs_work to needs_review

comment:25 in reply to: ↑ 22 ; follow-ups: Changed 4 years ago by tmonteil

Hi,

the third point was the minor one,

Ciao, Thierry

comment:26 in reply to: ↑ 25 Changed 4 years ago by ncohen

  • Description modified (diff)

comment:27 Changed 4 years ago by git

  • Commit changed from a372011eb60f2a3e6b01831430743a7421fd56f8 to 7a67d4dedbed50e2f3d492321b5a566a4c17027b

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

7a67d4dtrac #18126: Remove spkg-src

comment:28 in reply to: ↑ 25 Changed 4 years ago by ncohen

Hello,

In general applying patches is a good idea, but here we are patching an automatically generated file. I believe that it is safer to change it with a sed command (which just replaces some text with another one), as we would otherwise have to change the patch files (using the sed comand) whenever it changes.

I removed the script.

Nathann

comment:29 Changed 4 years ago by git

  • Commit changed from 7a67d4dedbed50e2f3d492321b5a566a4c17027b to a372011eb60f2a3e6b01831430743a7421fd56f8

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

comment:30 follow-up: Changed 4 years ago by ncohen

  • Description modified (diff)

Having a .tar.gz file seems to break the install scipts. I removed the last commit and set the ticket's description to its original text.

comment:31 Changed 4 years ago by git

  • Commit changed from a372011eb60f2a3e6b01831430743a7421fd56f8 to 7f8eda56a2f5b7f4f229d020a9d953308b9a7bc0

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

7f8eda5#18126 : use original .tar.gz source instead of custom .tar.bz2

comment:32 in reply to: ↑ 30 ; follow-up: Changed 4 years ago by tmonteil

  • Status changed from needs_review to needs_work

Replying to ncohen:

Having a .tar.gz file seems to break the install scipts.

I think this is because you did not update checksums.ini.

The installation works fine, but then, when i do sage -b, i got :

...
building 'sage.numerical.backends.coin_backend' extension
Executing 1 command (using 1 thread)
error: [Errno 13] Permission denied

Do you know what could be wrong ?

comment:33 in reply to: ↑ 32 ; follow-up: Changed 4 years ago by ncohen

Yo !

I think this is because you did not update checksums.ini.

I am an idiot T_T

The installation works fine, but then, when i do sage -b, i got :

...
building 'sage.numerical.backends.coin_backend' extension
Executing 1 command (using 1 thread)
error: [Errno 13] Permission denied

Do you know what could be wrong ?

No idea. What is the line that raises that? Are you the user with writing permissions on SAGE_ROOT and its subdirectories?

Nathann

comment:34 in reply to: ↑ 33 ; follow-up: Changed 4 years ago by tmonteil

Replying to ncohen:

No idea. What is the line that raises that?

The full output of sage -b command is:

scons: `install' is up to date.
Updating Cython code....
Enabling Cython debugging support
Finished Cythonizing, time: 14.40 seconds.
Discovering Python source code....
Discovered Python source, time: 0.23 seconds.
Cleaning up stale installed files....
- cleaning /opt/sagemath/sage-6.3/local/lib/python2.7/site-packages
- cleaning /opt/sagemath/sage-6.3/local/lib/site-python
- cleaning /opt/sagemath/sage-6.3/src/build/lib.linux-i686-2.7
Finished cleaning, time: 0.29 seconds.
running install
running build
running build_py
running build_ext
building 'sage.numerical.backends.coin_backend' extension
Executing 1 command (using 1 thread)
error: [Errno 13] Permission denied

Note that sage -i cbc ended with Finished installing cbc-2.9.4.spkg. I have been out of memory during sd66, i hope it did not mess up something on my directory.

Are you the user with writing permissions on SAGE_ROOT and its subdirectories?

Yes, i have only one user on my computer :)

comment:35 in reply to: ↑ 34 ; follow-up: Changed 4 years ago by fbissey

Replying to tmonteil:

Yes, i have only one user on my computer :)

But that doesn't mean the folder belongs to you, there are plenty of system class user out there - root in particular. Have you done anything as root with your install?

comment:36 in reply to: ↑ 35 Changed 4 years ago by tmonteil

Replying to fbissey:

Have you done anything as root with your install?

Not that i recall, but perhaps some chmod operation was not allowed when my disk was full. I double-checked that all files and subdirectories of SAGE_ROOT belong to my daily user (find ! -user 1000 leads to nothing). I removed all cbc-related code, and the problem is still there, so it is probably an unrelated issue, sorry for the noise, i will come back to the review once this is fixed.

comment:37 Changed 4 years ago by ncohen

  • Status changed from needs_work to needs_review

I set the ticket back to needs_review in the meantime. I have seen tickets go forgotten just because of their status :-P

Nathann

comment:38 Changed 4 years ago by vdelecroix

  • Status changed from needs_review to needs_work

The following worked well:

Could you update the ticket description?

Vincent

comment:39 follow-up: Changed 4 years ago by ncohen

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

comment:40 in reply to: ↑ 39 ; follow-up: Changed 4 years ago by vdelecroix

Replying to ncohen:

Note that the installation with .tgz did not work for me and I had to rename it .tar.gz.

Vincent

comment:41 in reply to: ↑ 40 Changed 4 years ago by ncohen

Note that the installation with .tgz did not work for me and I had to rename it .tar.gz.

I had not noticed. Could you make the modification so that I do not get it wrong again?

Nathann

comment:42 Changed 4 years ago by vdelecroix

  • Description modified (diff)

Running sage -t numerical/backends/coin_backend.pyx --optional Coin I got a lot of errors claiming that get_solver is not defined. Could you update the doctests in that ticket?

Vincent

comment:43 Changed 4 years ago by ncohen

you must run then with --optional=sage,coin. Otherwise ONLY the lines with a flag get executed.

Nathann

comment:44 Changed 4 years ago by vdelecroix

  • Reviewers set to David Coudert, Thierry Monteil, Vincent Delecroix
  • Status changed from needs_review to positive_review

Much better ;-)

comment:45 Changed 4 years ago by jdemeyer

  • Status changed from positive_review to needs_work

Please remove this message

ATTENTION: YOU HAVE TO REBUILD THE SAGE LIBRARY TO GET THIS TO WORK

After building a package, there is already

Warning: it might be needed to update the Sage library before
installed packages work: you should run 'make' from $SAGE_ROOT
before running Sage.

comment:46 Changed 4 years ago by git

  • Commit changed from 7f8eda56a2f5b7f4f229d020a9d953308b9a7bc0 to a3289854b1d4da2d1b126ecc788f17966482df04

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

a328985trac #18126: Review

comment:47 Changed 4 years ago by ncohen

  • Status changed from needs_work to positive_review

comment:48 follow-up: Changed 4 years ago by vbraun

Download link with the correct filename?

comment:49 in reply to: ↑ 48 Changed 4 years ago by ncohen

Download link with the correct filename?

wget 
http://www.coin-or.org/download/source/Cbc/Cbc-2.9.4.tgz -O cbc-2.9.4.tar.gz
Last edited 4 years ago by ncohen (previous) (diff)

comment:50 Changed 4 years ago by vbraun

Thanks, if you would also please integrate the renaming into the release management script?

comment:51 Changed 4 years ago by vbraun

Thanks, if you would also please integrate the renaming into the release management script?

comment:52 Changed 4 years ago by ncohen

Do you need a permanent url for the package with the right name?

Nathann

comment:53 Changed 4 years ago by vbraun

yes, that is preferred

comment:54 Changed 4 years ago by ncohen

What for? Don't you download it just once, to add it to Sage's mirrors?

Nathann

comment:55 Changed 4 years ago by vbraun

upstream might also modify their tarball while we are waiting for sage-6.6

comment:56 Changed 4 years ago by ncohen

We have the checksums, no worries.

Nathann

comment:57 Changed 4 years ago by vbraun

I'm not testing optional package checksums fyi

comment:58 Changed 4 years ago by ncohen

Would it be better for you if I re-wrote the spkg-src script that we had before? It would download the file and renames it for you.

Nathann

comment:59 Changed 4 years ago by ncohen

Volker?

comment:60 follow-up: Changed 4 years ago by vbraun

How's that going to help? I'm not going to manually run a different script every time some tarball gets updated.

comment:61 in reply to: ↑ 60 Changed 4 years ago by ncohen

How's that going to help? I'm not going to manually run a different script every time some tarball gets updated.

Well right now you have to download it manually [1]. They don't upload the tarballs on my server when they make a new release, and unfortunately you cannot download it directly yourself because the first letter is in upper case. Life is tough.

Nathann

[1] http://www.steinertriples.fr/ncohen/tmp/cbc-2.9.4.tar.gz

Last edited 4 years ago by ncohen (previous) (diff)

comment:62 Changed 4 years ago by vbraun

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