Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 525716 - dev-vcs/git-2.0.4 fails test t1304 in Docker due to missing USER
Summary: dev-vcs/git-2.0.4 fails test t1304 in Docker due to missing USER
Status: UNCONFIRMED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: [OLD] Development (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Robin Johnson
URL:
Whiteboard:
Keywords: PATCH
Depends on:
Blocks:
 
Reported: 2014-10-18 00:38 UTC by W. Trevor King
Modified: 2014-10-18 06:55 UTC (History)
1 user (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
Patch explicitly setting USER before running the tests. (git-test-user.patch,729 bytes, patch)
2014-10-18 00:39 UTC, W. Trevor King
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description W. Trevor King 2014-10-18 00:38:23 UTC
I'm not sure how others are avoiding this issue, but in Docker I'm failing t1304 with:

  not ok 3 - Objects creation does not break ACLs with restrictive umask
  #
  #               # SHA1 for empty blob
  #               check_perms_and_acl .git/objects/e6/9de29bb2d1d6434b8b29ae775ad8c2e48c5391
  #
  not ok 4 - git gc does not break ACLs with restrictive umask
  #
  #               git gc &&
  #               check_perms_and_acl .git/objects/pack/*.pack
  #

because USER isn't set.

Reproducible: Always

Steps to Reproduce:
1. $ docker run --rm -i -t wking/gentoo-portage:20141002 /bin/bash
2. # FEATURES=test emerge -v dev-vcs/git
Actual Results:  
…
failed test(s): t1304, t9903
…


Expected Results:  
Successful emerge.

The wking/gentoo-portage:20141002 is a stock amd64 stage3 with the same day's package tree.  I can post 'emerge --info' and such if you want more details.  See also the upstream patch [1].

[1]: http://thread.gmane.org/gmane.comp.version-control.git/258460
Comment 1 W. Trevor King 2014-10-18 00:39:37 UTC
Created attachment 386854 [details, diff]
Patch explicitly setting USER before running the tests.

For some reason, explicitly setting USER before calling emerge doesn't cut it.  This patch works though (coreutils gives us 'id').