Hi Petr, Petr Stodulka wrote:
Hi folks, I am looking at the #1357438 BZ about broken links to "How to*" doc files and I am thinking, about the best solution of this. Problem is with using of %doc macro, which moves/copies doc files to specific directories of each subpackage. However the Makefile expects that will be used just one directory, where all documentation will be included.It can be fixed basically in two ways:1) Do not use %doc macro and keep all Doc files under common directory, e.g. /usr/share/doc/git/ ignoring the sub-package that install specific doc files.2) Use sed for affected doc files to modify path correctly.The 1st method seems much better for me, because doc files will be together and in case of changes of doc files or another split/rename/merge of packages, it will be still OK. The 2nd method would provide incompatible solution in future when another changes in doc files will be provided or split of packages will be different.I haven't seen any requirement in packaging guidelines, that we have to put all files to specific directories bounded with specific subpackage, so why do not use '/usr/share/doc/git'?. The third option would be create symlink, but that solution seems ugly to me.What do you think? In case we will want to change filelist, I would prefer make this change in F26 yet too.
I also think that all the docs belong in /usr/share/doc/git.There was a thread on either devel or packaging a year or so ago regarding interations between using %doc in %files and manually placing files in %docdir. I don't know if that will come up here or not. It's easy enough to check the rpm contents after any changes to the file list though.
-- Todd ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ I don't mind arguing with myself. It's when I lose that it bothers me. -- Richard Powers
Attachment:
signature.asc
Description: PGP signature
_______________________________________________ devel mailing list -- devel@xxxxxxxxxxxxxxxxxxxxxxx To unsubscribe send an email to devel-leave@xxxxxxxxxxxxxxxxxxxxxxx
