wrong ownership of some files in git directory

[GoogleCodeExporter]

What steps will reproduce the problem?
1. Run the GUI git installer
2. Examine what was installed by doing cd /usr/local/git; ls -alR

What is the expected output? What do you see instead?

I expect the installed files to be owned by root. Instead, many of the 
installed files (e.g. Library directory and its contents, git-receive-pack, 
python2.5, etc.) are owned by user 502. I am user 501 so this makes no sense.

What version of the product are you using? On what operating system?

git-1.7.3.1-intel-leopard.dmg, on Snow Leopard

Original issue reported on code.google.com by mattne...@gmail.com on 20 Oct 2010 at 5:49

[GoogleCodeExporter]

Sigh, once again an artifact of Apple's poor excuse for a package builder.  
It's been extremely frustrating to work with because many settings will apply 
the first time you use them, but not the second time.  I have switched to 
hand-editing the manifest xml files but this is obviously not sustainable.

If anyone knows of a better packager please let me know... or if someone knows 
a better way to create installer pacakges.

Original comment by timchar...@gmail.com on 8 Dec 2010 at 7:18

• Changed state: Accepted

[GoogleCodeExporter]

Issue 59 has been merged into this issue.

Original comment by timchar...@gmail.com on 8 Dec 2010 at 7:18

[GoogleCodeExporter]

I apologize for the duplicates. But I don't really apologize, because they were 
due to an issue with Google Code's web interface, making it look my attempt to 
submit the bug the first two times had failed.

Original comment by mattne...@gmail.com on 8 Dec 2010 at 7:29

[GoogleCodeExporter]

I have looked at the xml I am feeding the packager, and it is CLEARLY telling 
it to package those files as root.  I have tested the installer, and my files 
are owned as root.  Can you download the latest package (1.7.3.3) and confirm 
if this is fixed?

Original comment by timchar...@gmail.com on 9 Dec 2010 at 10:18

[GoogleCodeExporter]

OMM, "mrclean" is user 502. I am user 501, "mattleopard".

hume:git mattleopard$ cd /usr/local/git; ls -alR | grep "mrclean"
lrwxr-xr-x  1 mrclean  wheel        3 Dec  9 14:48 git-receive-pack -> git
lrwxr-xr-x  1 mrclean  wheel        3 Dec  9 14:48 git-upload-archive -> git
lrwxr-xr-x    1 mrclean  wheel      13 Dec  9 14:48 git -> ../../bin/git
lrwxr-xr-x    1 mrclean  wheel      13 Dec  9 14:48 git-add -> ../../bin/git
lrwxr-xr-x    1 mrclean  wheel      13 Dec  9 14:48 git-annotate -> 
../../bin/git
lrwxr-xr-x    1 mrclean  wheel      13 Dec  9 14:48 git-apply -> ../../bin/git
lrwxr-xr-x    1 mrclean  wheel      13 Dec  9 14:48 git-archive -> ../../bin/git
lrwxr-xr-x    1 mrclean  wheel      13 Dec  9 14:48 git-bisect--helper -> 
../../bin/git
lrwxr-xr-x    1 mrclean  wheel      13 Dec  9 14:48 git-blame -> ../../bin/git
lrwxr-xr-x    1 mrclean  wheel      13 Dec  9 14:48 git-branch -> ../../bin/git
lrwxr-xr-x    1 mrclean  wheel      13 Dec  9 14:48 git-bundle -> ../../bin/git
lrwxr-xr-x    1 mrclean  wheel      13 Dec  9 14:48 git-cat-file -> 
../../bin/git
lrwxr-xr-x    1 mrclean  wheel      13 Dec  9 14:48 git-check-attr -> 
../../bin/git
lrwxr-xr-x    1 mrclean  wheel      13 Dec  9 14:48 git-check-ref-format -> 
../../bin/git
lrwxr-xr-x    1 mrclean  wheel      13 Dec  9 14:48 git-checkout -> 
../../bin/git
lrwxr-xr-x    1 mrclean  wheel      13 Dec  9 14:48 git-checkout-index -> 
../../bin/git
lrwxr-xr-x    1 mrclean  wheel      13 Dec  9 14:48 git-cherry -> ../../bin/git
lrwxr-xr-x    1 mrclean  wheel      13 Dec  9 14:48 git-cherry-pick -> 
../../bin/git
lrwxr-xr-x    1 mrclean  wheel      13 Dec  9 14:48 git-clean -> ../../bin/git
lrwxr-xr-x    1 mrclean  wheel      13 Dec  9 14:48 git-clone -> ../../bin/git
lrwxr-xr-x    1 mrclean  wheel      13 Dec  9 14:48 git-commit -> ../../bin/git
lrwxr-xr-x    1 mrclean  wheel      13 Dec  9 14:48 git-commit-tree -> 
../../bin/git
lrwxr-xr-x    1 mrclean  wheel      13 Dec  9 14:48 git-config -> ../../bin/git
lrwxr-xr-x    1 mrclean  wheel      13 Dec  9 14:48 git-count-objects -> 
../../bin/git
lrwxr-xr-x    1 mrclean  wheel      13 Dec  9 14:48 git-describe -> 
../../bin/git
lrwxr-xr-x    1 mrclean  wheel      13 Dec  9 14:48 git-diff -> ../../bin/git
lrwxr-xr-x    1 mrclean  wheel      13 Dec  9 14:48 git-diff-files -> 
../../bin/git
lrwxr-xr-x    1 mrclean  wheel      13 Dec  9 14:48 git-diff-index -> 
../../bin/git
lrwxr-xr-x    1 mrclean  wheel      13 Dec  9 14:48 git-diff-tree -> 
../../bin/git
lrwxr-xr-x    1 mrclean  wheel      13 Dec  9 14:48 git-fast-export -> 
../../bin/git
lrwxr-xr-x    1 mrclean  wheel      13 Dec  9 14:48 git-fetch -> ../../bin/git
lrwxr-xr-x    1 mrclean  wheel      13 Dec  9 14:48 git-fetch-pack -> 
../../bin/git
lrwxr-xr-x    1 mrclean  wheel      13 Dec  9 14:48 git-fmt-merge-msg -> 
../../bin/git
lrwxr-xr-x    1 mrclean  wheel      13 Dec  9 14:48 git-for-each-ref -> 
../../bin/git
lrwxr-xr-x    1 mrclean  wheel      13 Dec  9 14:48 git-format-patch -> 
../../bin/git
lrwxr-xr-x    1 mrclean  wheel      13 Dec  9 14:48 git-fsck -> ../../bin/git
lrwxr-xr-x    1 mrclean  wheel      13 Dec  9 14:48 git-fsck-objects -> 
../../bin/git
lrwxr-xr-x    1 mrclean  wheel      13 Dec  9 14:48 git-gc -> ../../bin/git
lrwxr-xr-x    1 mrclean  wheel      13 Dec  9 14:48 git-get-tar-commit-id -> 
../../bin/git
lrwxr-xr-x    1 mrclean  wheel      13 Dec  9 14:48 git-grep -> ../../bin/git
lrwxr-xr-x    1 mrclean  wheel      13 Dec  9 14:48 git-hash-object -> 
../../bin/git
lrwxr-xr-x    1 mrclean  wheel      13 Dec  9 14:48 git-help -> ../../bin/git
lrwxr-xr-x    1 mrclean  wheel      13 Dec  9 14:48 git-index-pack -> 
../../bin/git
lrwxr-xr-x    1 mrclean  wheel      13 Dec  9 14:48 git-init -> ../../bin/git
lrwxr-xr-x    1 mrclean  wheel      13 Dec  9 14:48 git-init-db -> ../../bin/git
lrwxr-xr-x    1 mrclean  wheel      13 Dec  9 14:48 git-log -> ../../bin/git
lrwxr-xr-x    1 mrclean  wheel      13 Dec  9 14:48 git-ls-files -> 
../../bin/git
lrwxr-xr-x    1 mrclean  wheel      13 Dec  9 14:48 git-ls-remote -> 
../../bin/git
lrwxr-xr-x    1 mrclean  wheel      13 Dec  9 14:48 git-ls-tree -> ../../bin/git
lrwxr-xr-x    1 mrclean  wheel      13 Dec  9 14:48 git-mailinfo -> 
../../bin/git
lrwxr-xr-x    1 mrclean  wheel      13 Dec  9 14:48 git-mailsplit -> 
../../bin/git
lrwxr-xr-x    1 mrclean  wheel      13 Dec  9 14:48 git-merge -> ../../bin/git
lrwxr-xr-x    1 mrclean  wheel      13 Dec  9 14:48 git-merge-base -> 
../../bin/git
lrwxr-xr-x    1 mrclean  wheel      13 Dec  9 14:48 git-merge-file -> 
../../bin/git
lrwxr-xr-x    1 mrclean  wheel      13 Dec  9 14:48 git-merge-index -> 
../../bin/git
lrwxr-xr-x    1 mrclean  wheel      13 Dec  9 14:48 git-merge-ours -> 
../../bin/git
lrwxr-xr-x    1 mrclean  wheel      13 Dec  9 14:48 git-merge-recursive -> 
../../bin/git
lrwxr-xr-x    1 mrclean  wheel      13 Dec  9 14:48 git-merge-subtree -> 
../../bin/git
lrwxr-xr-x    1 mrclean  wheel      13 Dec  9 14:48 git-merge-tree -> 
../../bin/git
lrwxr-xr-x    1 mrclean  wheel      13 Dec  9 14:48 git-mktag -> ../../bin/git
lrwxr-xr-x    1 mrclean  wheel      13 Dec  9 14:48 git-mktree -> ../../bin/git
lrwxr-xr-x    1 mrclean  wheel      13 Dec  9 14:48 git-mv -> ../../bin/git
lrwxr-xr-x    1 mrclean  wheel      13 Dec  9 14:48 git-name-rev -> 
../../bin/git
lrwxr-xr-x    1 mrclean  wheel      13 Dec  9 14:48 git-notes -> ../../bin/git
lrwxr-xr-x    1 mrclean  wheel      13 Dec  9 14:48 git-pack-objects -> 
../../bin/git
lrwxr-xr-x    1 mrclean  wheel      13 Dec  9 14:48 git-pack-redundant -> 
../../bin/git
lrwxr-xr-x    1 mrclean  wheel      13 Dec  9 14:48 git-pack-refs -> 
../../bin/git
lrwxr-xr-x    1 mrclean  wheel      13 Dec  9 14:48 git-patch-id -> 
../../bin/git
lrwxr-xr-x    1 mrclean  wheel      13 Dec  9 14:48 git-peek-remote -> 
../../bin/git
lrwxr-xr-x    1 mrclean  wheel      13 Dec  9 14:48 git-prune -> ../../bin/git
lrwxr-xr-x    1 mrclean  wheel      13 Dec  9 14:48 git-prune-packed -> 
../../bin/git
lrwxr-xr-x    1 mrclean  wheel      13 Dec  9 14:48 git-push -> ../../bin/git
lrwxr-xr-x    1 mrclean  wheel      13 Dec  9 14:48 git-read-tree -> 
../../bin/git
lrwxr-xr-x    1 mrclean  wheel      13 Dec  9 14:48 git-receive-pack -> 
../../bin/git
lrwxr-xr-x    1 mrclean  wheel      13 Dec  9 14:48 git-reflog -> ../../bin/git
lrwxr-xr-x    1 mrclean  wheel      13 Dec  9 14:48 git-remote -> ../../bin/git
lrwxr-xr-x    1 mrclean  wheel      13 Dec  9 14:48 git-replace -> ../../bin/git
lrwxr-xr-x    1 mrclean  wheel      13 Dec  9 14:48 git-repo-config -> 
../../bin/git
lrwxr-xr-x    1 mrclean  wheel      13 Dec  9 14:48 git-rerere -> ../../bin/git
lrwxr-xr-x    1 mrclean  wheel      13 Dec  9 14:48 git-reset -> ../../bin/git
lrwxr-xr-x    1 mrclean  wheel      13 Dec  9 14:48 git-rev-list -> 
../../bin/git
lrwxr-xr-x    1 mrclean  wheel      13 Dec  9 14:48 git-rev-parse -> 
../../bin/git
lrwxr-xr-x    1 mrclean  wheel      13 Dec  9 14:48 git-revert -> ../../bin/git
lrwxr-xr-x    1 mrclean  wheel      13 Dec  9 14:48 git-rm -> ../../bin/git
lrwxr-xr-x    1 mrclean  wheel      13 Dec  9 14:48 git-send-pack -> 
../../bin/git
lrwxr-xr-x    1 mrclean  wheel      13 Dec  9 14:48 git-shortlog -> 
../../bin/git
lrwxr-xr-x    1 mrclean  wheel      13 Dec  9 14:48 git-show -> ../../bin/git
lrwxr-xr-x    1 mrclean  wheel      13 Dec  9 14:48 git-show-branch -> 
../../bin/git
lrwxr-xr-x    1 mrclean  wheel      13 Dec  9 14:48 git-show-ref -> 
../../bin/git
lrwxr-xr-x    1 mrclean  wheel      13 Dec  9 14:48 git-stage -> ../../bin/git
lrwxr-xr-x    1 mrclean  wheel      13 Dec  9 14:48 git-status -> ../../bin/git
lrwxr-xr-x    1 mrclean  wheel      13 Dec  9 14:48 git-stripspace -> 
../../bin/git
lrwxr-xr-x    1 mrclean  wheel      13 Dec  9 14:48 git-symbolic-ref -> 
../../bin/git
lrwxr-xr-x    1 mrclean  wheel      13 Dec  9 14:48 git-tag -> ../../bin/git
lrwxr-xr-x    1 mrclean  wheel      13 Dec  9 14:48 git-tar-tree -> 
../../bin/git
lrwxr-xr-x    1 mrclean  wheel      13 Dec  9 14:48 git-unpack-file -> 
../../bin/git
lrwxr-xr-x    1 mrclean  wheel      13 Dec  9 14:48 git-unpack-objects -> 
../../bin/git
lrwxr-xr-x    1 mrclean  wheel      13 Dec  9 14:48 git-update-index -> 
../../bin/git
lrwxr-xr-x    1 mrclean  wheel      13 Dec  9 14:48 git-update-ref -> 
../../bin/git
lrwxr-xr-x    1 mrclean  wheel      13 Dec  9 14:48 git-update-server-info -> 
../../bin/git
lrwxr-xr-x    1 mrclean  wheel      13 Dec  9 14:48 git-upload-archive -> 
../../bin/git
lrwxr-xr-x    1 mrclean  wheel      13 Dec  9 14:48 git-var -> ../../bin/git
lrwxr-xr-x    1 mrclean  wheel      13 Dec  9 14:48 git-verify-pack -> 
../../bin/git
lrwxr-xr-x    1 mrclean  wheel      13 Dec  9 14:48 git-verify-tag -> 
../../bin/git
lrwxr-xr-x    1 mrclean  wheel      13 Dec  9 14:48 git-whatchanged -> 
../../bin/git
lrwxr-xr-x    1 mrclean  wheel      13 Dec  9 14:48 git-write-tree -> 
../../bin/git
drwxr-xr-x   3 mrclean  staff    102 Dec  9 14:48 Git Gui.app
drwxr-xr-x   3 mrclean  staff   102 Dec  9 14:48 .
drwxr-xr-x   5 mrclean  staff   170 Dec  9 14:07 Contents
drwxr-xr-x  5 mrclean  staff  170 Dec  9 14:07 .
drwxr-xr-x  3 mrclean  staff  102 Dec  9 14:48 ..
-rw-r--r--  1 mrclean  staff  903 Dec  9 14:07 Info.plist
drwxr-xr-x  3 mrclean  staff  102 Dec  9 14:07 MacOS
drwxr-xr-x  4 mrclean  staff  136 Dec  9 14:07 Resources
drwxr-xr-x  3 mrclean  staff     102 Dec  9 14:07 .
drwxr-xr-x  5 mrclean  staff     170 Dec  9 14:07 ..
-rwxr-xr-x  1 mrclean  staff  164944 Dec  9 14:07 Wish
drwxr-xr-x  4 mrclean  staff    136 Dec  9 14:07 .
drwxr-xr-x  5 mrclean  staff    170 Dec  9 14:07 ..
drwxr-xr-x  3 mrclean  staff    102 Dec  9 14:07 Scripts
-rw-r--r--  1 mrclean  staff  28866 Dec  9 14:07 git-gui.icns
drwxr-xr-x  3 mrclean  staff  102 Dec  9 14:07 .
drwxr-xr-x  4 mrclean  staff  136 Dec  9 14:07 ..
-rw-r--r--  1 mrclean  staff  588 Dec  9 14:07 AppMain.tcl

Original comment by mattne...@gmail.com on 9 Dec 2010 at 10:51

[GoogleCodeExporter]

Ah, I see.  It's not all of the files, just some.  I'll fix for the next 
release.

Original comment by timchar...@gmail.com on 9 Dec 2010 at 11:45

[GoogleCodeExporter]

fixed by 
https://github.com/timcharper/git_osx_installer/commit/8bfd924014c2484653c90cf68
875d5d94e5b7aa0

Original comment by timchar...@gmail.com on 10 Dec 2010 at 12:16

[GoogleCodeExporter]

I have redeployed 1.7.3.3.  I can verify on my end that all files are now owned 
as root.

Original comment by timchar...@gmail.com on 10 Dec 2010 at 12:22

• Changed state: Fixed

[GoogleCodeExporter]

That fixes it, thanks!

Original comment by mattne...@gmail.com on 10 Dec 2010 at 2:02