Hi Vasyl,
On Mon, Feb 01, 2021 at 02:02:28PM +0000, Vasyl Gello wrote:
> Hi Ian, Guido!
> 
> > I'm going to reply to your next part in general terms. I hope Guido will 
> > agree with me.
> 
> Thanks for explanation! :) NMU looks not that scary thing for me!
> There is no need for NMU since Guido is here, but noted for the future.
> 
> Guido, how is it best for you to review patches?
> 
> * Attaching to bugs
> * Using Salsa cloned repo in my personal space
> 
> As for breakage, the best solution I see here is to introduce new gbp.conf 
> parameter
> like USCAN_SYMLINK={rename|symlink} plus check and warn the user if uscan's 
> behavior
> is overridden by environment variable somewhere.

I think the best way is to figure out if we can't just drop the option
and have users configure it via `~/.devscripts.conf`. The reason i did 
(unfortunately)
not act on it yet is that i didn't get around to test all the
possibilities like `--tarball-dir`, multiple tarballs, etc.
I went back in the git history the other day and didn't spot anything
obvious but didn't want to make the change close to Debian release
without thorough testing. So if you want to check that out, that'd be
great.

We have some network based tests see `GBP_NETWORK_TESTS` so if you want
to add some testing the different import cases with `--uscan` based
tests. We could cook up some watch files that pull in things from a
stable location.

Cheers,
 -- Guido

> 
> What do you think? If the approach is fine, let me craft a PR!
> -- 
> Vasyl Gello
> ==================================================
> Certified SolidWorks Expert
> 
> Mob.:+380 (98) 465 66 77
> 
> E-Mail: vasek.ge...@gmail.com
> 
> Skype: vasek.gello
> ==================================================
> 호랑이는 죽어서 가죽을 남기고 사람은 죽어서 이름을 남긴다

Reply via email to