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

nauty upstream source tarball has setgid on top-level directory #20870

Closed
embray opened this issue Jun 24, 2016 · 6 comments
Closed

nauty upstream source tarball has setgid on top-level directory #20870

embray opened this issue Jun 24, 2016 · 6 comments

Comments

@embray
Copy link
Contributor

embray commented Jun 24, 2016

The top-level directory in the current source tarball for the nauty spkg has the setgid bit set on it for some reason.

This probably is not intentional and may be harmless in general, but it is causing confusion for Cygwin's permission handling, making the package unbuildable.

CC: @orlitzky

Component: packages: standard

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

@embray embray added this to the sage-7.3 milestone Jun 24, 2016
@embray
Copy link
Contributor Author

embray commented Jul 25, 2016

comment:1

Should this be reported upstream? Who made this tarball? This is absolutely never clear....

@dimpase
Copy link
Member

dimpase commented Jul 25, 2016

comment:2

Please do report; however, the tarball needed to be renamed, see #19919, but not re-packed IIRC.

@jdemeyer
Copy link

comment:3

Can't this be fixed on the level of sage-uncompress-spkg? Especially since that script already fixes some permission issues.

@embray
Copy link
Contributor Author

embray commented Jul 26, 2016

comment:4

It is fixed in sage-uncompress-spkg; doesn't mean it shouldn't be fixed here too.

@jdemeyer
Copy link

comment:5

Right, but then it's not so relevant for Sage.

@orlitzky
Copy link
Contributor

comment:6

This was not really our problem, but in any case, they've stopped doing it in the latest tarball:

https://pallini.di.uniroma1.it/nauty27r3.tar.gz

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

5 participants