Re: [Quilt-dev] patch - list all changed files in all applied patches

[Dean Roehrich]

>From:  Andreas Gruenbacher <address@hidden>

>I like it. A worthwhile extension might be to also allow passing a range of 
>patches as in the diff command. For that, the first patch to include needs to 
>be specified, e.g.,
>
>       --combine {patch|-}   =>   first patch
>
>With ``--combine -'' being be the same as ``-a''.

Perhaps.  I'll take a look at it.


>Not sure whether putting patch names in brackets is necessarily a good idea --
> 
>having no delimiters would at least make scripting easier.

Sure.  I was looking for human-readable output in this case and the delimiters
helped me, or so I think.  In the "-a -l" case, separating the filename from
the patchname with only a single space doesn't help my eyes, and using a tab
is questionable as patch names and path names become longer.

So let's play with it a bit...


# separate with space
$ quilt files -al | sort +1
...snip...
[export-backing_dev_info] drivers/block/ll_rw_blk.c
[kdb-common] drivers/char/keyboard.c
[kdb-common] drivers/char/sn_serial.c
[kdb-common] drivers/serial/8250.c
[dmapi-enable] fs/Kconfig
[jfs-dmapi-2.6.7-rc1.patch] fs/Kconfig
[jfs-dmapi-2.6.8-rc1-additional] fs/Kconfig
[xfs-debug] fs/Kconfig
[xfs-modules] fs/Kconfig
[dmapi-enable] fs/Makefile
[unified_dmapi] fs/dmapi/Makefile
[unified_dmapi] fs/dmapi/Makefile-linux-2.4
...snip...


# separate with tabs
$ quilt files -al | sort +1
...snip...
[export-backing_dev_info]       drivers/block/ll_rw_blk.c
[kdb-common]    drivers/char/keyboard.c
[kdb-common]    drivers/char/sn_serial.c
[kdb-common]    drivers/serial/8250.c
[dmapi-enable]  fs/Kconfig
[jfs-dmapi-2.6.7-rc1.patch]     fs/Kconfig
[jfs-dmapi-2.6.8-rc1-additional]        fs/Kconfig
[xfs-debug]     fs/Kconfig
[xfs-modules]   fs/Kconfig
[dmapi-enable]  fs/Makefile
[unified_dmapi] fs/dmapi/Makefile
[unified_dmapi] fs/dmapi/Makefile-linux-2.4
...snip...


# separate with space, but no delimiters
$ quilt files -al | sort +1
...snip...
export-backing_dev_info drivers/block/ll_rw_blk.c
kdb-common drivers/char/keyboard.c
kdb-common drivers/char/sn_serial.c
kdb-common drivers/serial/8250.c
dmapi-enable fs/Kconfig
jfs-dmapi-2.6.7-rc1.patch fs/Kconfig
jfs-dmapi-2.6.8-rc1-additional fs/Kconfig
xfs-debug fs/Kconfig
xfs-modules fs/Kconfig
dmapi-enable fs/Makefile
unified_dmapi fs/dmapi/Makefile
unified_dmapi fs/dmapi/Makefile-linux-2.4
...snip...


Hmmm...my eyes need delimiters.  How about adding -v for the [patchname]
delimiters?

For the "-a -v" case I could go either way--with or without delimiters.

# with delimiters
$ quilt files -av
...snip...
[xfs-modules]
  fs/Kconfig
[xfs-debug]
  fs/Kconfig
[dmapi-enable]
  Documentation/filesystems/xfs.txt
  fs/Kconfig
  fs/Makefile
[export-backing_dev_info]
  drivers/block/ll_rw_blk.c
[docs-update]
  Documentation/filesystems/xfs.txt
...snip...


# without delimiters
$ quilt files -av
...snip...
xfs-modules
  fs/Kconfig
xfs-debug
  fs/Kconfig
dmapi-enable
  Documentation/filesystems/xfs.txt
  fs/Kconfig
  fs/Makefile
export-backing_dev_info
  drivers/block/ll_rw_blk.c
docs-update
  Documentation/filesystems/xfs.txt
...snip...


Dean