By default, Bugzilla does not search the list of RESOLVED bugs.
You can force it to do so by putting the upper-case word ALL in front of your search query, e.g.: ALL tdelibs
We recommend searching for bugs this way, as you may discover that your bug has already been resolved and fixed in a later release.
Bug 350 - Build issue: Source tarball permissions are goofy
Summary: Build issue: Source tarball permissions are goofy
Status: RESOLVED FIXED
Alias: None
Product: TDE
Classification: Unclassified
Component: system (show other bugs)
Version: R14.0.0 [Trinity]
Hardware: All All
: P1 blocker
Assignee: Timothy Pearson
URL:
Depends on:
Blocks:
 
Reported: 2010-10-06 17:38 CDT by Darrell
Modified: 2013-04-07 16:36 CDT (History)
4 users (show)

See Also:
Compiler Version:
TDE Version String:
Application Version:
Application Name:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Darrell 2010-10-06 17:38:51 CDT
The 3.5.12 source tarballs contain odd file and directory permissions. Many files have 640 permissions and directories 750. Although packages normally are built with root permissions, these oddball permissions prevent browsing and studying the uncompressed archive as a normal user.

Minimally, for future releases, the mechanism for creating the tarballs should be reviewed to ensure the uncompressed archive is viewable by all users. If possible, repost the 3.5.12 tarballs with appropriate file and directory permissions.
Comment 1 Darrell 2012-01-12 21:33:17 CST
Same problem with 3.5.13. :( Files and directories should be world readable within the tarballs.

I build my packages in a chroot running as root. The build location is readable by my normal user account. When I am testing patches and the patches fail the build script halts. I can't view or browse files in the build area as normal user, which I prefer because my chroot build environment is command line only. Konqueror and Kate are so much more pleasant to use. :) I have to chown the files from within the build environment before I can browse the build area.

I can do that from within my build script, but releasing the tarballs that way is unusual. I have never seen this before. Every package I have built through the years has world readable permissions.

This is not a bug per se, but sure is inconvenient. The problem is tarballs only, not my local GIT tree. I'll add this item to the etherpad check list. :)
Comment 2 Timothy Pearson 2012-01-31 14:08:36 CST
Raising to BLOCKER as R14.0 should have this issue fixed.  This will require a simple update of the release generation scripts.
Comment 3 Darrell 2012-06-13 11:32:20 CDT
Could this report be closed as resolved? We have the problem itemized in our etherpad pre-release check list.
Comment 4 Darrell 2012-10-16 21:43:02 CDT
Looks like this slipped through the hoops with 3.5.13.1. File permissions are slavek:users rather than root:root.
Comment 5 Slávek Banko 2012-11-21 18:48:02 CST
I examined the current GIT tree and did not find any file or folder that should be with permissions 640 or 750. If future packages will be packaged under the user instead of the root, then this problem has not come. Is that so?
Comment 6 Darrell 2012-11-21 19:26:11 CST
I suppose as long as all files in any tarball are world readable by any user then half the original problem disappears.

Seems to me ownership in the tarballs should be root:root, but if not they should be something like trinity:trinity or trinity:users, or something like that.
Comment 7 Timothy Pearson 2013-04-07 01:33:30 CDT
This has been fixed in the release creation scripts on the TDE servers.

Specifically, the following commands will be executed prior to each tarball being created:
chown root *
chgrp root *
chmod -R g+r,o+r *

Slavek, you should probably do the same in your SRU generation process.
Comment 8 Slávek Banko 2013-04-07 04:15:04 CDT
For creating tarballs I now use create_tarball included in the 'scripts'. In this is the setting owner and group already solved by another way - using tar parameters: --owner=root --group=users
Comment 9 Timothy Pearson 2013-04-07 16:36:34 CDT
(In reply to comment #8)
> For creating tarballs I now use create_tarball included in the 'scripts'. In
> this is the setting owner and group already solved by another way - using tar
> parameters: --owner=root --group=users

OK.  Is the 'users' group GID guaranteed to be the same across all *NIX systems?  If not, then root should probably be used to avoid undefined behaviour on the target system.