Opened 8 years ago

Closed 8 years ago

#11664 closed defect (fixed)

polybori files not world-readable

Reported by: jdemeyer Owned by: tbd
Priority: blocker Milestone: sage-4.7.1
Component: packages: standard Keywords:
Cc: Merged in: sage-4.7.1.rc2
Authors: Jeroen Demeyer Reviewers: Alexander Dreyer
Report Upstream: Fixed upstream, in a later stable release. Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Description (last modified by jdemeyer)

Similar to #11660, the polybori files are not world-readable.

Reported upstream: polybori-discuss

New spkg: http://boxen.math.washington.edu/home/jdemeyer/spkg/polybori-0.7.0.p4.spkg

Change History (7)

comment:1 Changed 8 years ago by jdemeyer

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

comment:2 Changed 8 years ago by jdemeyer

  • Report Upstream changed from N/A to Not yet reported upstream; Will do shortly.

comment:3 Changed 8 years ago by jdemeyer

  • Description modified (diff)
  • Report Upstream changed from Not yet reported upstream; Will do shortly. to Reported upstream. Little or no feedback.

comment:4 Changed 8 years ago by AlexanderDreyer

  • Report Upstream changed from Reported upstream. Little or no feedback. to Fixed upstream, in a later stable release.

Thanks for pointing this out. Upstream, the tarball generation (for distribution) will now fix the permissions accordingly, see https://bitbucket.org/brickenstein/polybori/changeset/2e495ed559e7/raw/polybori-2e495ed559e7.diff

comment:5 Changed 8 years ago by AlexanderDreyer

Note, that sage -pkg needs to be fixed also.

comment:6 Changed 8 years ago by AlexanderDreyer

  • Status changed from needs_review to positive_review

The package builds, installs and passes the relevant tests. I' fine withthe differences and merges it to #11261 (soon to #11575, too).

comment:7 Changed 8 years ago by jdemeyer

  • Merged in set to sage-4.7.1.rc2
  • Resolution set to fixed
  • Reviewers set to Alexander Dreyer
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.