Re: [PATCH] Add pagure SCM/forge

[ng0]

Leo Famulari <address@hidden> writes:

> On Sun, Dec 11, 2016 at 06:12:26PM +0000, ng0 wrote:
>> [PATCH 01/11] gnu: Add python-flask-wtf.
>> [PATCH 02/11] gnu: Add python-flask-multistatic.
>> [PATCH 03/11] gnu: Add python-kitchen.
>> [PATCH 04/11] gnu: Add python-munch.
>> [PATCH 05/11] gnu: Add python-pygit2.
>> [PATCH 06/11] gnu: Add python2-openid.
>> [PATCH 07/11] gnu: Add python-openid-cla.
>> [PATCH 08/11] gnu: Add python-openid-teams.
>> [PATCH 09/11] gnu: Add python-straight.plugin.
>> [PATCH 10/11] gnu: Add python-trollius-redis.
>> [PATCH 11/11] gnu: Add pagure.
>
> Awesome!
>
> I sent a few comments about licenses. Can you check those and send
> updated patches if necessary?
>
> I also noticed a few packages that pull their source from GitHub even
> though it is available on PyPi, like python-openid-teams:
>
> https://pypi.python.org/pypi/python-openid-teams
>
> Is there a reason for that? If not, can you make sure to use the PyPi
> source code instead?
>
> Thanks!
>

Wasn't it like when the source is on github or somewhere else
upstream, we should loose the dependency on pypi infrastructure
and use that upstream? That's why I changed source where
possible.

I can send the updates when I'm back from Berlin, late Wednesday
- Thursday'ish

Thanks for your review.
-- 
♥Ⓐ  ng0  | ng0.chaosnet.org