File::Find::Rule is introduced as a dependency in the work for bug 7167. However, since it's indirectly invoked via C4::Installer, that module must be present for Makefile.PL to compile during installation. I suggest making a new convention that it be possible to run Makefile.PL without requiring non-core Perl modules. F::F::Rule doesn't introduce functionality that is not available to File::Find, just some (admittedly useful) syntactical sugar, so one possibility would be cutting out that dependency entirely. If that's not preferred, it at least should not be required for Makefile.PL to run.
Created attachment 14311 [details] [review] Bug 9328, 7167: Remove the File::Find::Rule dependency F::F::Rule doesn't introduce functionality that is not available to File::Find, just some (admittedly useful) syntactical sugar.
To apply on top of patches for Bug 7167
Created attachment 16118 [details] [review] Bug 9328, 7167: Remove the File::Find::Rule dependency F::F::Rule doesn't introduce functionality that is not available to File::Find, just some (admittedly useful) syntactical sugar.
Created attachment 17715 [details] [review] Bug 9328, 7167: Remove the File::Find::Rule dependency F::F::Rule doesn't introduce functionality that is not available to File::Find, just some (admittedly useful) syntactical sugar. Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
The status of 7167 is not clear to me. That makes testing this patch even harder.. What I read here, sounds good to me.
This patch does no longer apply: Apply? [y]es/[n]o/[e]dit/[v]iew patch/[a]ccept all y Applying: Bug 9328, 7167: Remove the File::Find::Rule dependency fatal: sha1 information is lacking or useless (C4/Update/Database.pm). Repository lacks necessary blobs to fall back on 3-way merge. Cannot fall back to three-way merge. Patch failed at 0001 Bug 9328, 7167: Remove the File::Find::Rule dependency When you have resolved this problem run "git am --resolved". If you would prefer to skip this patch, instead run "git am --skip". To restore the original branch and stop patching run "git am --abort".
(In reply to Katrin Fischer from comment #6) > This patch does no longer apply: This patch depends on bug 7167. Back to Signed Off.
I am not sure how to proceed with this as I think it currently can't be tested. I am moving it to 'BLOCKED' for now.
(In reply to Katrin Fischer from comment #8) > I am not sure how to proceed with this as I think it currently can't be > tested. I am moving it to 'BLOCKED' for now. I know bug 7167 is not a priority for Galen but this path can be tested. Especially because it is a small and easy patch.