search

sagemath / sage

Main repository of SageMath
https://www.sagemath.org
Other
1.45k stars 482 forks source link

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

Closed embray closed 2 years ago

embray commented 8 years ago

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 commented 8 years ago
comment:1

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

dimpase commented 8 years ago
comment:2

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

jdemeyer commented 8 years ago
comment:3

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

embray commented 8 years ago
comment:4

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

jdemeyer commented 8 years ago
comment:5

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

orlitzky commented 2 years ago
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