Opened 7 years ago

Closed 7 years ago

#15436 closed defect (fixed)

Cksum uses tabs instead of spaces, breaking sage-spkg's regex

Reported by: vbraun Owned by:
Priority: major Milestone: sage-6.0
Component: build Keywords:
Cc: Merged in:
Authors: Volker Braun Reviewers:
Report Upstream: N/A Work issues:
Branch: u/vbraun/ticket/15436 (Commits) Commit: 992ea564f1702e387fcf918b12a790f38140a835
Dependencies: Stopgaps:

Description

If neither shasum nor md5sum/md5 is available, then we fall back to cksum for package checksums. This only during adventures in retro computing, i.e., Solaris. This patch fixes the regex to work even with the broken Solaris sed.

Change History (6)

comment:1 Changed 7 years ago by vbraun

  • Branch set to u/vbraun/ticket/15436
  • Created changed from 11/19/13 03:36:33 to 11/19/13 03:36:33
  • Modified changed from 11/19/13 03:36:33 to 11/19/13 03:36:33

comment:2 Changed 7 years ago by vbraun

  • Authors set to Volker Braun
  • Commit set to 992ea564f1702e387fcf918b12a790f38140a835
  • Status changed from new to needs_review

comment:3 Changed 7 years ago by vbraun

Works on Skynet/mark now....

comment:4 Changed 7 years ago by robertwb

  • Status changed from needs_review to positive_review

So, not even \s is supported. In that case, LGTM.

comment:5 Changed 7 years ago by jdemeyer

  • Milestone changed from sage-5.13 to sage-6.0

comment:6 Changed 7 years ago by vbraun

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