On Wed, 24 Oct 2018 at 09:12, Daniel Vetter <[email protected]> wrote: > > On Tue, Oct 23, 2018 at 04:33:21PM +0100, Emil Velikov wrote: > > From: Emil Velikov <[email protected]> > > > > Seems like some of my existing URLs had a trailing forward slash. > > This confused url_to_remote(), resulting in adding the exact same repo > > again. > > > > Signed-off-by: Emil Velikov <[email protected]> > > --- > > dim | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/dim b/dim > > index 93e90a4f96d0..71a681278cbc 100755 > > --- a/dim > > +++ b/dim > > @@ -192,7 +192,7 @@ function url_to_remote # url [url ...] > > fi > > > > for url; do > > - remote=$(git remote -v | grep -m 1 "$url (" | cut -f 1) > > + remote=$(git remote -v | grep -m 1 "$url/* (" | cut -f 1) > > ? instead of *, or do you have entire piles of trailing slashes? Otherwise > lgtm, makes sense.
Just one, although it made sense to strip any arbitrary amount. v2 coming shortly, will let you guys decide which one is better. -Emil _______________________________________________ dim-tools mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/dim-tools
