Re: [PATCH dim 2/2] dim: Add add-link command

[Jani Nikula <jani.nikula@linux.intel.com>]

On Wed, 15 Mar 2017, Jani Nikula <jani.nikula@linux.intel.com> wrote:
> On Tue, 14 Mar 2017, ville.syrjala@linux.intel.com wrote:
>> From: Ville Syrjälä <ville.syrjala@linux.intel.com>
>>
>> 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@linux.intel.com>
>> ---
>>  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@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx