From: Jani Nikula <jani.nikula@intel.com>
To: Andrzej Hajda <a.hajda@samsung.com>, dri-devel@lists.freedesktop.org
Cc: Daniel Vetter <daniel.vetter@ffwll.ch>,
Marek Szyprowski <m.szyprowski@samsung.com>,
Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>
Subject: Re: [maintainer-tools PATCH] dim: fix remote repository lookup function
Date: Thu, 28 Sep 2017 17:51:18 +0300 [thread overview]
Message-ID: <87d16a7u6x.fsf@intel.com> (raw)
In-Reply-To: <6fab9388-5ac5-c04d-3e06-3ef3f958b2b7@samsung.com>
On Wed, 27 Sep 2017, Andrzej Hajda <a.hajda@samsung.com> wrote:
> On 25.08.2017 13:31, Andrzej Hajda wrote:
>> dim finds remote name by matching repository urls, but different users
>> requires different protocols/paths for remotes (ssh/git/https). Current
>> code incorrectly translates provided url to alternatives, the patch
>> fixes it.
>>
>> Signed-off-by: Andrzej Hajda <a.hajda@samsung.com>
>
> Gently ping.
Apologies for the delay. I had a hard time understanding all the sed
trickery you have there. Then I decided the right solution is to be able
to configure multiple URLs in nightly.conf for each repo in
drm_tip_repos.
Patch follows adding this support to dim, untested because I need to run
now. Unfortunately we can't update nightly.conf before dim users have
upgraded to the patched version, because otherwise their setups would
break.
BR,
Jani.
--
Jani Nikula, Intel Open Source Technology Center
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
next prev parent reply other threads:[~2017-09-28 14:51 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <CGME20170825113227eucas1p2beec67e2f2a805ccd34cf7bf71ed4ec1@eucas1p2.samsung.com>
2017-08-25 11:31 ` [maintainer-tools PATCH] dim: fix remote repository lookup function Andrzej Hajda
2017-09-27 9:58 ` Andrzej Hajda
2017-09-28 14:51 ` Jani Nikula [this message]
2017-09-28 14:51 ` [dim PATCH] dim: allow a space separated list of URLs for each repo in drm_tip_repos Jani Nikula
2017-10-03 6:15 ` Jani Nikula
2017-10-03 8:01 ` Daniel Vetter
2017-10-09 10:30 ` Jani Nikula
2017-10-09 10:37 ` Daniel Stone
2017-10-11 19:22 ` Jani Nikula
2017-10-10 8:10 ` Andrzej Hajda
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=87d16a7u6x.fsf@intel.com \
--to=jani.nikula@intel.com \
--cc=a.hajda@samsung.com \
--cc=b.zolnierkie@samsung.com \
--cc=daniel.vetter@ffwll.ch \
--cc=dri-devel@lists.freedesktop.org \
--cc=m.szyprowski@samsung.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.