Ticket #7909 (closed defect: fixed)

Opened 3 years ago

Last modified 3 years ago

Replace $MKDIR with 'mkdir' in sage-spkg

Reported by: drkirkby Owned by: GeorgSWeber
Priority: major Milestone: sage-4.3.1
Component: build Keywords:
Cc: jsp Work issues:
Report Upstream: N/A Reviewers: Jaap Spies
Authors: David Kirkby Merged in: sage-4.3.1.rc0
Dependencies: Stopgaps:

Description (last modified by jsp) (diff)

sage-spkg has $MKDIR in one place, which will cause problems with an updated 'sage-env' which no longer defines MKDIR.

I'm attaching a copy of the revised sage-spkg, and also a Mercurial patch.

Attachments

sage-spkg Download (12.1 KB) - added by drkirkby 3 years ago.
A complete copy of the revised sage-env
sage-spkg-remove-MKDIR.patch Download (547 bytes) - added by drkirkby 3 years ago.
Mercurial patch for sage-env

Change History

Changed 3 years ago by drkirkby

A complete copy of the revised sage-env

Changed 3 years ago by drkirkby

Mercurial patch for sage-env

comment:1 Changed 3 years ago by drkirkby

  • Status changed from new to needs_review

comment:2 follow-up: ↓ 3 Changed 3 years ago by jsp

  • Reviewers set to Jaap Spies

Don't know whether the name 'sage-spkg' is any good. Did you mean to attach the new sage-env?

The patch is simple and looks good.

Jaap

comment:3 in reply to: ↑ 2 Changed 3 years ago by jsp

  • Description modified (diff)

Replying to jsp:

Don't know whether the name 'sage-spkg' is any good. Did you mean to attach the new sage-env?

The patch is simple and looks good.

Jaap

It is really in sage-spkg!! I changed the description.

Waiting for the mercurial patch.

Jaap

comment:4 Changed 3 years ago by jsp

  • Status changed from needs_review to positive_review

The "mercurial patch" is ok. sage-spkg looks good. So positive review.

Jaap

comment:5 Changed 3 years ago by rlm

  • Status changed from positive_review to closed
  • Resolution set to fixed
  • Merged in set to sage-4.3.1.rc0
Note: See TracTickets for help on using tickets.