Bug 11692 – can't set file attributes for std.zip.ArchiveMember

Status
RESOLVED
Resolution
FIXED
Severity
regression
Priority
P2
Component
phobos
Product
D
Version
D2
Platform
All
OS
Linux
Creation time
2013-12-05T15:30:00Z
Last change time
2014-01-15T11:41:39Z
Assigned to
nobody
Creator
code

Comments

Comment #0 by code — 2013-12-05T15:30:15Z
To set the external (OS specific) file attributes one also has to set it's format which is stored in the upper byte of madeVersion. With https://github.com/D-Programming-Language/phobos/pull/1697 madeVersion became a read-only property so this is no longer possible. This is needed to store zip files with executable flag or directories on Posix.
Comment #1 by github-bugzilla — 2013-12-09T03:19:07Z
Commits pushed to master at https://github.com/D-Programming-Language/phobos https://github.com/D-Programming-Language/phobos/commit/463e5d803bf01637487494c93d00e23e8cffbc64 fix Issue 11692 - can't set file attributes for std.zip.ArchiveMember - Document fileAttributes and set the internal _madeVersion according to the encode attribute type. - deprecate direct access to the externalAttributes field https://github.com/D-Programming-Language/phobos/commit/aae7c341d2ccfb60b2bdb93abc421e3558a70626 Merge pull request #1751 from dawgfoto/std_zip [install,regression] fix Issue 11692 - can't set file attributes for std.zip.ArchiveMember
Comment #2 by github-bugzilla — 2013-12-19T07:01:33Z
Commit pushed to 2.065 at https://github.com/D-Programming-Language/phobos https://github.com/D-Programming-Language/phobos/commit/67933fda951ccd8735eb66ea223980ae7175bc11 Merge pull request #1751 from dawgfoto/std_zip [install,regression] fix Issue 11692 - can't set file attributes for std.zip.ArchiveMember
Comment #3 by github-bugzilla — 2014-01-15T11:41:39Z