Ticket #11661 (closed defect: fixed)

Opened 22 months ago

Last modified 22 months ago

gap files not world-readable

Reported by: jdemeyer Owned by: tbd
Priority: blocker Milestone: sage-4.7.1
Component: packages: standard Keywords:
Cc: Work issues:
Report Upstream: Reported upstream. Developers acknowledge bug. Reviewers: Leif Leonhardy
Authors: Jeroen Demeyer Merged in: sage-4.7.1.rc2
Dependencies: Stopgaps:

Description (last modified by jdemeyer) (diff)

Similar to #11660, the gap spkg has some files which are not world-reable:

102075160    4 -rwxr-x---   1 jdemeyer jdemeyer     2100 Dec 18  2008 gap-4.4.12.p5/src/terminfo/x/xterm
102060573    4 -rwxr-x---   1 jdemeyer jdemeyer     1533 Dec 18  2008 gap-4.4.12.p5/src/terminfo/c/cygwin
102075162    4 -rwxr-x---   1 jdemeyer jdemeyer     1885 Dec 18  2008 gap-4.4.12.p5/src/terminfo/r/rxvt

New spkg, changing permissions and removing obsolete Debian files:  http://boxen.math.washington.edu/home/jdemeyer/spkg/gap-4.4.12.p6.spkg

Change History

comment:1 Changed 22 months ago by jdemeyer

  • Status changed from new to needs_review
  • Description modified (diff)
  • Authors set to Jeroen Demeyer

comment:2 Changed 22 months ago by jdemeyer

  • Report Upstream changed from N/A to Reported upstream. Little or no feedback.

Reported upstream by email to [support@…]

comment:3 Changed 22 months ago by jdemeyer

  • Report Upstream changed from Reported upstream. Little or no feedback. to Reported upstream. Developers acknowledge bug.

Answer from upstream:

Dear Jeroen,

Thank you for telling us about this. We will take care about this in 
the GAP 4.5.2 (aka 2nd beta) on which we're working now. 

Actually, these files only make sense in the Windows binary distribution,
so the source distro does not need to have them at all.

Best wishes,
Alexander

comment:4 follow-up: ↓ 6 Changed 22 months ago by leif

  • Status changed from needs_review to positive_review
  • Reviewers set to Leif Leonhardy

Well, although the first commit message contains the second, the second doesn't contain a ticket number, and the last changeset isn't tagged, positive review, since the permissions are "fixed" now, though this IMHO was a non-issue (cf. upstream reply), not worth a blocker.

The spkg is a mess anyway.

comment:5 Changed 22 months ago by jdemeyer

  • Status changed from positive_review to closed
  • Resolution set to fixed
  • Merged in set to sage-4.7.1.rc2

comment:6 in reply to: ↑ 4 Changed 22 months ago by jdemeyer

Replying to leif:

this IMHO was a non-issue (cf. upstream reply), not worth a blocker.

Probably true, but I did not realize this when making this ticket. I simply checked all non-world-readable files in the Sage install. I can proudly say that now all installed files all world-readable (at least if the umask allows this).

Note: See TracTickets for help on using tickets.