Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Closed
vbraun opened this issue Nov 19, 2013 · 6 comments
Closed

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

vbraun opened this issue Nov 19, 2013 · 6 comments

Comments

@vbraun
Copy link
Member

vbraun commented Nov 19, 2013

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.

Component: build

Author: Volker Braun

Branch/Commit: u/vbraun/ticket/15436 @ 992ea56

Reviewer: Robert Bradshaw

Issue created by migration from https://trac.sagemath.org/ticket/15436

@vbraun vbraun added this to the sage-5.13 milestone Nov 19, 2013
@vbraun
Copy link
Member Author

vbraun commented Nov 19, 2013

Branch: u/vbraun/ticket/15436

@vbraun
Copy link
Member Author

vbraun commented Nov 19, 2013

Commit: 992ea56

@vbraun
Copy link
Member Author

vbraun commented Nov 19, 2013

Author: Volker Braun

@vbraun
Copy link
Member Author

vbraun commented Nov 19, 2013

comment:3

Works on Skynet/mark now....

@robertwb
Copy link
Contributor

comment:4

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

@slel
Copy link
Member

slel commented May 11, 2021

Reviewer: Robert Bradshaw

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants