Changes between Initial Version and Version 19 of Ticket #10970


Ignore:
Timestamp:
09/08/11 22:55:27 (10 years ago)
Author:
leif
Comment:

I still think removing pipestatus for no reason at this point is simply superfluous.

The patch should IMHO at least test for pipestatus's presence, and issue an appropriate error message (with instructions how to fix this) if it doesn't exist; though quite late in the upgrade process, i.e. after all updated packages have been downloaded, of course.

P.S.: We could even attempt to download it (with e.g. curl or wget) from the script. Or install the root repository "manually" in advance on upgrading, of course from the updated spkg/install, since the other scripts are those of the old version.

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #10970

    • Property Status changed from new to positive_review
    • Property Cc jdemeyer leif added
    • Property Summary changed from Remove pipestatus from spkg/install to Do not generate pipestatus from spkg/install
    • Property Keywords sd32 added
    • Property Reviewers changed from to Mariah Lenox
  • Ticket #10970 – Description

    initial v19  
    11The `$SAGE_ROOT/spkg/install` has a here document that creates `pipestatus`. With the sage root repository, this is no longer needed as `pipestatus` is tracked by the repository. The patch removes it from `$SAGE_ROOT/spkg/install`.
     2
     3----
     4
     5Apply [attachment:trac_10970_do_not_generate_pipestatus.patch] to the Sage '''root repository''' (in `$SAGE_ROOT`).