Re: [ptxdist] [PATCH] ptxd_make_world_patchin: Allow git-ptx-* scripts to be overwritten

[Thorsten Scherer <t.scherer@eckelmann.de>]

Hello Michael,

On Wed, Jun 21, 2023 at 10:55:12AM +0200, Michael Olbrich wrote:
> On Mon, Jun 19, 2023 at 03:02:10PM +0200, Thorsten Scherer wrote:
> > This patch enables the user to make local modifications to
> > git-ptx-patches and git-ptx-refresh-tags-editor inside the BSP.
> 
> Out of curiosity: Do you have real local changes, or is this just to be
> able to backport fixes in the future?

We just bumped to ptxdist-2023.05.0 and I had to modify
ptxd_make_world_patchin.sh to be able to benefit from b6be1a59d.
Without this patch ptx-patches aborts with "Failed to find tag 'base' in
the series".

> Regards,
> Michael

Regards
Thorsten

> > Signed-off-by: Thorsten Scherer <t.scherer@eckelmann.de>
> > ---
> >  scripts/lib/ptxd_make_world_patchin.sh | 12 ++++++++++--
> >  1 file changed, 10 insertions(+), 2 deletions(-)
> > 
> > diff --git a/scripts/lib/ptxd_make_world_patchin.sh b/scripts/lib/ptxd_make_world_patchin.sh
> > index 026c36e95..209fc6954 100644
> > --- a/scripts/lib/ptxd_make_world_patchin.sh
> > +++ b/scripts/lib/ptxd_make_world_patchin.sh
> > @@ -62,6 +62,14 @@ ptxd_make_world_patchin_apply_git_init()
> >      local git_dir
> >      git_dir="$(git rev-parse --git-dir 2> /dev/null)" || true
> >  
> > +    local git_ptx_patches
> > +    ptxd_in_path PTXDIST_PATH_SCRIPTS git-ptx-patches
> > +    git_ptx_patches="${ptxd_reply}"
> > +
> > +    local git_ptx_refresh_tags_editor
> > +    ptxd_in_path PTXDIST_PATH_SCRIPTS git-ptx-refresh-tags-editor
> > +    git_ptx_refresh_tags_editor="${ptxd_reply}"
> > +
> >      # is already git repo?
> >      if [ "${git_dir}" != ".git" ]; then
> >  	echo "patchin: git: initializing repository"
> > @@ -70,8 +78,8 @@ ptxd_make_world_patchin_apply_git_init()
> >  	__git commit -q -m "initial commit" --author="ptxdist-${PTXDIST_VERSION_FULL} <ptxdist@pengutronix.de>" &&
> >  	__git tag "${pkg_pkg//\~/-}" &&
> >  	__git tag base &&
> > -	__git config alias.ptx-patches "!${PTXDIST_TOPDIR}/scripts/git-ptx-patches \"\${@}\"" &&
> > -	__git config sequence.editor "${PTXDIST_TOPDIR}/scripts/git-ptx-refresh-tags-editor" &&
> > +	__git config alias.ptx-patches "!${git_ptx_patches} \"\${@}\"" &&
> > +	__git config core.editor "${git_ptx_refresh_tags_editor}" &&
> >  	__git config diff.renames false &&
> >  	__git config core.abbrev 12 &&
> >  	__git config core.autocrlf false &&
> > -- 
> > 2.38.0
> > 
> > 
> > 
> 
> -- 
> Pengutronix e.K.                           |                             |
> Steuerwalder Str. 21                       | http://www.pengutronix.de/  |
> 31137 Hildesheim, Germany                  | Phone: +49-5121-206917-0    |
> Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |