darcs

Patch 358 resolve issue1815: generate tarballs with the right permissions

Title resolve issue1815: generate tarballs with the right permissions
Superseder Nosy List tux_rocker
Related Issues
Status accepted Assigned To
Milestone

Created on 2010-08-22.19:17:35 by tux_rocker, last changed 2011-05-10.18:36:41 by darcswatch. Tracked on DarcsWatch.

Files
File name Status Uploaded Type Edit Remove
resolve-issue1815.dpatch tux_rocker, 2010-08-22.19:17:35 text/plain
See mailing list archives for discussion on individual patches.
Messages
msg12268 (view) Author: tux_rocker Date: 2010-08-22.19:17:35
'runghc Setup.lhs sdist' generates tarballs with whacky permissions (Cabal
issue 627 [1]). 'cabal sdist' gets the permissions right but does not allow 
you
to have automatically generated modules (Cabal issue 729 [2]). We have
automatically generated modules so we can't use 'cabal sdist'. So the release
scripts now untars the tarball, adjust permissions, and tars it up again.

[1]: http://hackage.haskell.org/trac/hackage/ticket/627
[2]: http://hackage.haskell.org/trac/hackage/ticket/729

Reinier
Attachments
msg12270 (view) Author: mornfall Date: 2010-08-22.20:46:32
Hi,

On Sun, Aug 22, 2010 at 07:17:35PM +0000, Reinier Lamers wrote:
> 'runghc Setup.lhs sdist' generates tarballs with whacky permissions (Cabal
> issue 627 [1]). 'cabal sdist' gets the permissions right but does not allow 
> you
> to have automatically generated modules (Cabal issue 729 [2]). We have
> automatically generated modules so we can't use 'cabal sdist'. So the release
> scripts now untars the tarball, adjust permissions, and tars it up again.

> +# fix tarball permissions (Cabal bug 627)
> +# we can't use 'cabal sdist' as a workaround because we have our own sdist hook
> +find $packagename -type f -exec chmod 644 {} +
> +find $packagename -type d -exec chmod 755 {} +
> +
> +tar czf $tarballname $packagename
> +
> +test -f $tarballname
> +cp -f $tarballname $wd/dist/$tarballname
> +
> +# okay, permissions fixed, test if we can build and test from the tarball
Should we rm -rf $packagename here? (out of sheer paranoia)
> +tar xvzf $wd/dist/$tarballname
>  cd $packagename
>  
>  runghc Setup configure --user -ftype-witnesses -ftest

Otherwise, looks OK. Please push at your convenience, I still don't have my
laptop (until Tuesday).

Yours,
   Petr.
msg12370 (view) Author: kowey Date: 2010-08-29.08:14:05
This wasn't being tracked by darcswatch
History
Date User Action Args
2010-08-22 19:17:35tux_rockercreate
2010-08-22 19:18:10darcswatchsetdarcswatchurl: http://darcswatch.nomeata.de/repo_http:__darcs.net_.html#bundle-89de0c305be77e99fddb876c0cdfe7c5df5660fc
2010-08-22 20:46:32mornfallsetmessages: + msg12270
2010-08-29 08:14:05koweysetstatus: needs-review -> accepted
messages: + msg12370
2011-05-10 18:36:41darcswatchsetdarcswatchurl: http://darcswatch.nomeata.de/repo_http:__darcs.net_.html#bundle-89de0c305be77e99fddb876c0cdfe7c5df5660fc -> http://darcswatch.nomeata.de/repo_http:__darcs.net_reviewed.html#bundle-89de0c305be77e99fddb876c0cdfe7c5df5660fc