Commit graph

23 commits

Author SHA1 Message Date
Patrick Uiterwijk
e592912070 Actually, Todd was right and the failed text was invalid. Let's ignore
Signed-off-by: Patrick Uiterwijk <puiterwijk@redhat.com>
2018-07-19 00:00:59 +02:00
Patrick Uiterwijk
84141c0895 Pretty sure we want to encode
Signed-off-by: Patrick Uiterwijk <puiterwijk@redhat.com>
2018-07-18 23:44:21 +02:00
Pierre-Yves Chibon
3b1e225f73 Add missing get_default_branch 2018-05-03 10:40:13 +02:00
Todd Zullinger
476ca97e74 git (post-receive-alternativearch): use unicode for git output
The commit message and diff is stored in `full_change` as a string, via
`read_output()`.  This is passed to `TEXT` which is a unicode string.

When a commit message or diff contains non-ascii characters we get:

    UnicodeDecodeError: 'ascii' codec can't decode byte ...

Encode git output returned from `read_output()` as a unicode string and
define `full_change` as unicode for completeness.

Fixes #6040.
2018-01-19 16:40:08 +01:00
Dan Horák
333a155ba9 post-receive-alternativearch - catch removals too
Currently the patterns can catch ExcludeArch & co additions and changes
in the spec file, but not a clean removal.
2017-07-25 10:57:44 +02:00
Pierre-Yves Chibon
82fb1996dd Move to src.fp.o which is https 2017-04-18 14:29:30 +02:00
Pierre-Yves Chibon
4b45f4620c Make the email sent a little prettier 2017-04-18 14:24:39 +02:00
Pierre-Yves Chibon
dc5394c189 Looks like all we need is a new line 2017-04-18 14:22:38 +02:00
Pierre-Yves Chibon
678e30e837 When sending alternative arch excluding emails, include the full change
Fixes https://pagure.io/fedora-infrastructure/issue/5923
2017-04-18 14:09:28 +02:00
Pierre-Yves Chibon
737f6c5b38 Escape the %ifarch and %ifnarch otherwise python is unhappy 2017-01-16 22:14:45 +01:00
Pierre-Yves Chibon
29a58c2460 Improve the subject of the email sent upon commit 2017-01-16 10:29:15 +01:00
Pierre-Yves Chibon
083901a312 Fix detecting %ifarch/%ifnarch upon commit and improve the email content 2017-01-16 10:24:30 +01:00
Pierre-Yves Chibon
7f0f327c3d Fix the ExcludeArch pattern and add two more 2017-01-13 11:58:05 +01:00
Pierre-Yves Chibon
a6fdbaeff6 Send all the notifications about package excluding some arch to the arch-excludes list 2017-01-10 16:33:15 +01:00
Pierre-Yves Chibon
fbeb42e3ca Layout fix 2016-11-01 11:14:47 +01:00
Pierre-Yves Chibon
d531c54bec Properly format the diffs sent 2016-11-01 11:03:56 +01:00
Pierre-Yves Chibon
7d3112e440 Correctly store the diffs 2016-11-01 10:58:44 +01:00
Pierre-Yves Chibon
54b2e3303a Include the lines changed/edited in the email sent 2016-11-01 10:58:44 +01:00
Pierre-Yves Chibon
8fa913c9c3 Let's look like the fedmsg hook do 2016-11-01 10:51:33 +01:00
Pierre-Yves Chibon
18002fb6c8 Turn off debugging for now 2016-11-01 10:38:43 +01:00
Pierre-Yves Chibon
7029868c06 Looks like we're having duplicated .git in there 2016-11-01 10:33:58 +01:00
Pierre-Yves Chibon
b007fb8249 For now send all the notifs to me 2016-11-01 10:27:34 +01:00
Pierre-Yves Chibon
4e153f8912 Rename the hook post-receive-secondaryarch into post-receive-alternativearch 2016-10-28 10:59:16 +02:00
Renamed from roles/git/hooks/files/post-receive-secondaryarch (Browse further)