Re: creating github pull requests in pkg-perl-tools

Hey!

Sorry for taking awhile to respond. I read this on my phone but forgot to respond :)

My goal with the repository was to use it as a mirror for pkg-perl (i.e. write a script to clone everything from our git repos, then push a copy to GitHub). Pull Requests would have to be merged manually and pushed to git.debian.org, and GitHub would detect this and mark the PR as closed (or else I'd have to do it manually). This would make it easier for people to find and submit patches to Debian.

So my initial goal wasn't to use it for upstream repos, though if the team agrees that's a better use of the org, we can certainly do so, I don't really have meaningful objections there (especially since I don't have working code) :)

On Sat, May 28, 2016 at 3:03 PM, gregor herrmann <gregoa@debian.org> wrote:

On Sat, 28 May 2016 18:34:55 +0200, Alex Muntada wrote:

> On the other hand, using the debian-perl organization in GitHub
> to fork the upstream repos and create the pull requests feels
> like the right thing to do. But maybe people would like to use
> their own GitHub accounts instead. Anyway, I'd like to use
> debian-perl organitzation for testing at least, so I'd appreciate
> an invitation to join.

What advantage do you see for using the org to fork the repos rather than your personal account? It seems confusing to use debian-perl for upstream code (rather, at least to me, it makes more sense to use it for downstream - our packaging code), but I might be the only person that thinks this.

I think that using the debian-perl organization in GitHub is an
excellent idea.
I don't know jawnsy's ideas when creating it but this seems a good
use case. Can you two please coordinate regarding its usage?

On Sat, 28 May 2016 17:55:36 +0000, Damyan Ivanov wrote:

> I tried the following "one-liner" (after mr up) and got no potential
> patches for forwarding.
>
> grep -i github */META.json \
> | cut -f1 -d/ | uniq \
> | while read d; do \
> if [ -d $d/debian/patches ]; then \
> find $d/debian/patches -name '*.patch' \
> -o -name '*.diff' \
> | while read f; do \
> grep -L -v '^Forwarded:' "$f"; \
> done; \
> fi; \
> done

\o/ Yay!
All patches (for upstream distributions using Github) forwarded! :)

Cheers,
gregor

--
.''`. Homepage https://info.comodo.priv.at/ - OpenPGP key 0xBB3A68018649AA06
: :' : Debian GNU/Linux user, admin, and developer - https://www.debian.org/
`. `' Member of VIBE!AT & SPI, fellow of the Free Software Foundation Europe
`- NP: Peter Jones: Let's Drink To The King

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1

iQJ8BAEBCgBmBQJXShW+XxSAAAAAAC4AKGlzc3Vlci1mcHJAbm90YXRpb25zLm9w
ZW5wZ3AuZmlmdGhob3JzZW1hbi5uZXREMUUxMzE2RTkzQTc2MEE4MTA0RDg1RkFC
QjNBNjgwMTg2NDlBQTA2AAoJELs6aAGGSaoGzTYQAMVPLLsOT2jD+qaLvE1aOCsM
MK+JoYrZ7MGnq11G90ZLfvlLxw1Rc4OKwkYQ0z44vL7XbW/GV8rV48EP4BbE3XWE
GCEKKninnk0tUZY0Z2N3MaS5ZC4MfyszIOjXYHDqzrGdYz573KvssAE486f1+y/f
Cy2/Ci4OFMr8Igfo/5MFzkR34wQqsOzFgf+Y7qxZuGGWyvRFVkX4x1mpeTwhsJRe
gheh/kEGVQrhpParljwkuWTRqW+VEo/OAa3t0p/lSb/8UuK70cggXJqkVWygPgz4
kECJsckNut+Raw2J/Ip+RXmiK9+SzXxbYuRKMK8hmgKQtPLMi4s7AfKgUJQzIKaY
uPFkBy37G1zqW8bFclNKwisOaCUjKx5T2boFM1X7ghLnVO0O3rIoNRlFq3/ybX6w
mwFKhKJvlFgRBbNlhAkwunYdpUPiK9+mt6vfJ8e8ioxLfhaoW8IWRtNLDsKOC1je
NB59vm6PjBiGJMxk9fycslk41oQfDrYB7VCk8yF4J8CkgNu4WS1TgZBU0NRPijFB
tI+fjHaG3cPGgn6sJMWe0sA88rAHXsyFTfhHoMMXM+wkUUssCR7Uz+erKnL3nMvV
k391hENJOVCIHkKe9tDOnE+/li5Mi/7vmjkxew9EIsjr5ERhZh6NGljcOEruEPU8
cQE6oiKn7xyqckgMGS2q
=Nmqy
-----END PGP SIGNATURE-----

Cheers,

Jonathan