#6594 UnicodeDecodeError in /usr/share/git-core/post-receive-alternativearch on git push
Closed: Duplicate Opened by tmz.

When pushing changes to the git package, I received the following traceback from the /usr/share/git-core/post-receive-alternativearch script:

$ git push 
Counting objects: 28, done.
Delta compression using up to 8 threads.
Compressing objects: 100% (28/28), done.
Writing objects: 100% (28/28), 6.43 KiB | 2.14 MiB/s, done.
Total 28 (delta 18), reused 0 (delta 0)
remote: Emitting a message to the fedmsg bus.
remote: * Publishing information for 9 commits
remote: * Notifying alternative-arch people
remote: Traceback (most recent call last):
remote:   File "/usr/share/git-core/post-receive-alternativearch", line 206, in <module>
remote:     run_as_post_receive_hook()
remote:   File "/usr/share/git-core/post-receive-alternativearch", line 198, in run_as_post_receive_hook
remote:     full_change=full_change
remote: UnicodeDecodeError: 'ascii' codec can't decode byte 0xc2 in position 17399: ordinal not in range(128)
remote: Sending to redis to log activity and send commit notification emails
remote: Detailed log of new commits:
remote: 
remote: 
remote: * commit ebf9643f6d6269e489831f8ee9ed1ca86cc493d5
remote: * Author: Todd Zullinger <tmz@pobox.com>
remote: * Date:   Fri Dec 22 19:14:31 2017 -0500
remote: * 
remote: * Re-enable grep tests on s390x
remote: * 
remote: * The tests appear to work reliably now, after a number of test builds. If
remote: * not, this is easily reverted.  (The newly added test output may also
remote: * help determine the root cause.)
remote: * 
remote: * commit 84015970a03269ff18d59184f2493c9d6688349f
remote: * Author: Todd Zullinger <tmz@pobox.com>
remote: * Date:   Fri Dec 22 17:09:04 2017 -0500
remote: * 
remote: * Skip 'git svn branch' tests which fail intermittently
remote: * 
remote: * All tests which call 'git svn branch' fail intermittently with SIGSEGV
remote: * in the subversion bindings.  The failures are not remotely consistent.
remote: * Until we can enable shell tracing (-x) in the test suite by default to
remote: * attempt to further debug the failure, disable these tests.
remote: * 
remote: * commit 62688d0cccb1e4e0eb8d1c3b2d80806528ffe741
remote: * Author: Todd Zullinger <tmz@pobox.com>
remote: * Date:   Fri Dec 22 16:41:13 2017 -0500
remote: * 
remote: * Run tests in parallel by default on Fedora
remote: * 
remote: * An option to enable parallel test runs was added in 6dc6285 ("Improve
remote: * test suite coverage", 2017-11-10).  After further improvements it seems
remote: * to be reliable for use in koji builds.
remote: * 
remote: * It is still disabled for EPEL builds because make lacks support for the
remote: * -O (--output-sync) option.  Without this option the output is much less
remote: * readable.
remote: * 
remote: * In the future we may be able to move to using 'prove' as the default
remote: * test harness which might allow us to run in parallel without requiring
remote: * the -O option.
remote: * 
remote: * commit db2eb1dae4e86a198ac34200573035e8218a288f
remote: * Author: Todd Zullinger <tmz@pobox.com>
remote: * Date:   Thu Nov 30 17:46:45 2017 -0500
remote: * 
remote: * Avoid failures in svnserve tests when run in parallel
remote: * 
remote: * Setting SVNSERVE_PORT enables several tests which require a local
remote: * svnserve daemon to be run (in t9113 & t9126).  The tests share setup of
remote: * the local svnserve via `start_svnserve()`.  The function uses svnserve's
remote: * `--listen-once` option, which causes svnserve to accept one connection
remote: * on the port, serve it, and exit.  When running the tests in parallel
remote: * this fails if one test tries to start svnserve while the other is still
remote: * running.
remote: * 
remote: * Use the test number as the svnserve port (similar to httpd tests) to
remote: * avoid port conflicts.  Set GIT_TEST_SVNSERVE to enable these tests.
remote: * 
remote: * The change should make it into 2.16 or 2.17.  Until then, setting
remote: * GIT_TEST_SVNSERVE will disable the svnserve tests.  There are only a few
remote: * tests affected and this allows more testing with parallel test runs, so
remote: * it's a reasonable trade.  Once an upstream release with these changes
remote: * arrives, we'll begin running these tests without any further changes.
remote: * 
remote: * commit 15d075cb09ab8b9b2621a82db9ece8493829b88f
remote: * Author: Todd Zullinger <tmz@pobox.com>
remote: * Date:   Wed Dec 20 13:24:12 2017 -0500
remote: * 
remote: * Mark git-core-docs package noarch
remote: * 
remote: * Nothing arch-specific should be included in the git-core-docs package.
remote: * 
remote: * commit 675b3bad861fef5bbf2e813f4927cade79c891af
remote: * Author: Todd Zullinger <tmz@pobox.com>
remote: * Date:   Wed Dec 20 12:57:25 2017 -0500
remote: * 
remote: * Fix builds using '--without docs'
remote: * 
remote: * Use '%bcond_without docs' and the corresponding %{with ...} syntax.  Fix
remote: * a number of areas where builds failed due to broken %if conditions.
remote: * 
remote: * commit 340e1bd03dd74e7d6079f48369060182812d3389
remote: * Author: Todd Zullinger <tmz@pobox.com>
remote: * Date:   Wed Dec 20 09:20:11 2017 -0500
remote: * 
remote: * Remove second make invocation for doc build/install
remote: * 
remote: * In c3202fd ("Use prebuilt documentation on EL-5, where asciidoc is too
remote: * old", 2013-01-04) the make targets to build and install the
remote: * documentation were split into separate make calls to handle prebuilt
remote: * docs on EL-5 where asciidoc was too old to build the documentation.
remote: * 
remote: * Support for EL-5 was removed in 903d8f3 ("Remove EL-5 and old Fedora
remote: * conditionals", 2017-07-22), making the separate make calls unneeded.
remote: * 
remote: * commit 53fdd5334a2afc8e7c63790e667bbf64cd8049c9
remote: * Author: Todd Zullinger <tmz@pobox.com>
remote: * Date:   Fri Dec 1 21:16:54 2017 -0500
remote: * 
remote: * Use %autosetup macro to unpack and patch source
remote: * 
remote: * Using %autosetup frees us from having to manage %patch entries.  It does
remote: * require that all patches use the same strip level (-p1 in our case).
remote: * There is also a mildly annoying bug in EL-6¹ which chokes on the patch
remote: * invocation if a blank line doesn't follow %autosetup.  A comment should
remote: * prevent any problems.
remote: * 
remote: * The %autosetup -S option is deliberately avoided, to prevent needless
remote: * bootstrapping problems.
remote: * 
remote: * ¹ https://bugzilla.redhat.com/1310704
remote: * 
remote: * commit 2e7d4595e521114fa6b11974249db8722b04b72f
remote: * Author: Todd Zullinger <tmz@pobox.com>
remote: * Date:   Thu Nov 30 16:40:38 2017 -0500
remote: * 
remote: * Include verbose logs in build output for 'make test' failures
remote: * 
remote: * When building in koji or copr and a test fails, the verbose output from
remote: * the tests can be very useful in determining the cause.  Print the output
remote: * from failing tests, borrowing heavily from the upstream code¹ used when
remote: * running tests in Travis CI².
remote: * 
remote: * This could be done inline in %check rather than in a separate script.
remote: * The downside is that rpm would include each command invocation of the
remote: * loop in the output, which adds a lot of useless text to the already
remote: * copious build output.
remote: * 
remote: * ¹ https://git.kernel.org/pub/scm/git/git.git/plain/ci/print-test-failures.sh
remote: * ² https://travis-ci.org/git/git
To ssh://pkgs.fedoraproject.org/rpms/git
   4079ad1..ebf9643  master -> master

This was reported in https://pagure.io/fedora-infrastructure/issue/6040

Closing this one in favor of that one. ;)

Thanks for reporting tho!

Metadata Update from @kevin:
- Issue close_status updated to: Duplicate
- Issue status updated to: Closed (was: Open)

Thanks for the pointer. I did search for unicodedecodeerror (with and without capitalization) before I filed it and got no results. I guess the search is a bit lacking. :/

Metadata