On Wed, 15 Mar 2017, Jani Nikula <jani.nikula@xxxxxxxxxxxxxxx> wrote:
> On Tue, 14 Mar 2017, ville.syrjala@xxxxxxxxxxxxxxx wrote:
>> From: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx>
>>
>> Add the "add-link" command so that you can add the Link: tag to
>> patches that failed to apply directly.
>>
>> Signed-off-by: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx>
>> ---
>> dim | 39 +++++++++++++++++++++++++++++++++++++++
>> 1 file changed, 39 insertions(+)
>>
>> diff --git a/dim b/dim
>> index 4110642b2f4a..b44d087b19ac 100755
>> --- a/dim
>> +++ b/dim
>> @@ -664,6 +664,45 @@ function dim_apply_branch
>> eval $DRY $DIM_POST_APPLY_ACTION
>> }
>>
>> +dim_alias_ll=add-link
>> +function dim_add_link
>> +{
>> + local branch=$1
>> + shift
>> + local file=`mktemp`
>
> Same complaints about locals as in previous patch.
>
>> +
>> + assert_branch $branch
>> + assert_repo_clean
>> +
>> + cat > $file
>> +
>> + local message_id=$(message_get_id $file)
rm -f $file
>> +
>> + if [ -n $message_id ]; then
>> + dim_commit_add_tag "Link: http://patchwork.freedesktop.org/patch/msgid/$message_id";
>> + else
>> + echo "No message-id found in the patch file."
>
> Please use echoerr to print to stderr.
>
>> + fi
>> +}
>> +
>> +dim_alias_lq=add-link-queued
>> +function dim_add_link_queued
>> +{
>> + dim_add_link drm-intel-next-queued "$@"
>> +}
>> +
>> +dim_alias_lf=add-link-fixes
>> +function dim_add_link_fixes
>> +{
>> + dim_add_link drm-intel-fixes "$@"
>> +}
>> +
>> +dim_alias_lnf=add-link-next-fixes
>> +function dim_add_link_next_fixes
>> +{
>> + dim_add_link drm-intel-next-fixes "$@"
>> +}
>> +
>
> I'm thinking this one's so rarely used that the aliases are not
> needed. Short aliases will make dim bash completion less useful, as the
> completion automatically completes all dim commands and aliases.
>
>> dim_alias_aq=apply-queued
>> function dim_apply_queued
>> {
--
Jani Nikula, Intel Open Source Technology Center
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
