Hello Franck,

note I didn't test your patch, only read through.  I didn't even read
the prose part.  I assume you only changed the formating!?

> diff --git a/create-help.sh b/create-help.sh
> index fbd9a09..b1e8a2a 100755
> --- a/create-help.sh
> +++ b/create-help.sh
> @@ -7,11 +7,11 @@ if [ $# -ne 1 ] ;. then
>       exit 1
>  fi
>  
> +TG_CMD=`echo $1 | tr '[:lower:]' '[:upper:]'`
> +
>  < README awk '
> -     BEGIN { incommand = 0; }
> -     /^tg '"$1"'$/ { incommand = 1; next; }
> -     /^~/ { next; } # Ignore the title underlines.
> -     /^[^\t]/ { incommand = 0; next; }
> +     /^TG-/ { if (incommand) { incommand = 0; } }
> +     /^TG-'"$TG_CMD"'\(1\)$/ { incommand = 1; }
>       { if (incommand) { print $0; } }
>  '  > tg-"$1".txt
>  
> diff --git a/debian/control b/debian/control
> index 35f6761..596e71a 100644
> --- a/debian/control
> +++ b/debian/control
> @@ -4,6 +4,7 @@ Priority: optional
>  Maintainer: Debian TopGit maintainers <pkg-top...@teams.debian.net>
>  Uploaders: martin f. krafft <madd...@debian.org>, Uwe Kleine-König 
> <u.kleine-koe...@pengutronix.de>
>  Build-Depends: debhelper (>= 6), quilt
> +Build-Depends-Indep: asciidoc, xmlto
hhhm.  I didn't start with using Build-Depends:, but as topgit is an
arch=all package, I wonder if most if not all dependencies should be
Build-Depends-Indep:.  Martin?

> --- a/debian/rules
> +++ b/debian/rules
> @@ -5,19 +5,28 @@
>  
>  TG_BRANCHES="fixes/ensure-worktree debian/locations"
>  
> +TG_MANS := $(patsubst %.sh,%.1,$(wildcard tg-*.sh))
> +
Does the package still build after 

        $ touch tg-astillundocumentedcommand.sh

?

> diff --git a/debian/topgit.manpages b/debian/topgit.manpages
> new file mode 100644
> index 0000000..98712e7
> --- /dev/null
> +++ b/debian/topgit.manpages
> @@ -0,0 +1,10 @@
> +tg-create.1
> +tg-delete.1
> +tg-depend.1
> +tg-import.1
> +tg-info.1
> +tg-mail.1
> +tg-patch.1
> +tg-remote.1
> +tg-summary.1
> +tg-update.1
What about autogenerating this file?

Best regards and thanks for your efforts
Uwe

-- 
Pengutronix e.K.                              | Uwe Kleine-König            |
Industrial Linux Solutions                    | http://www.pengutronix.de/  |



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org

Reply via email to