FS#49682 - [ocaml-findlib] Wrong manpage installation path

Attached to Project: Community Packages
Opened by Alif (alive4ever) - Sunday, 12 June 2016, 07:03 GMT
Last edited by Alexander F. Rødseth (xyproto) - Monday, 11 July 2016, 17:50 GMT
Task Type Bug Report
Category Packages
Status Closed
Assigned To Alexander F. Rødseth (xyproto)
Architecture All
Severity Low
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 1
Private No

Details

Description:
Section 3 and 5 of findlib manpages doesn't exist.


Additional info:
* ocaml-findlib 1.6.2-1
* $ pacman -Ql ocaml-findlib
...
ocaml-findlib /usr/share/man/man3 /
ocaml-findlib /usr/share/man/man3 /build/
ocaml-findlib /usr/share/man/man3 /build/ocaml-findlib/
ocaml-findlib /usr/share/man/man3 /build/ocaml-findlib/pkg/
ocaml-findlib /usr/share/man/man3 /build/ocaml-findlib/pkg/ocaml-findlib/
ocaml-findlib /usr/share/man/man3 /build/ocaml-findlib/pkg/ocaml-findlib/usr/
ocaml-findlib /usr/share/man/man3 /build/ocaml-findlib/pkg/ocaml-findlib/usr/share/
ocaml-findlib /usr/share/man/man3 /build/ocaml-findlib/pkg/ocaml-findlib/usr/share/man/
ocaml-findlib /usr/share/man/man3 /build/ocaml-findlib/pkg/ocaml-findlib/usr/share/man/man5/
...


Steps to reproduce:
Build ocaml-findlib from Arch Linux PKGBUILD.
List the resulting pkg.tar.xz files (tar -tf ocaml-findlib-1.6.2-1-x86_64.pkg.tar.xz).
Note that section 3 and 5 of manual pages don't exist. The resulting manpage structure should be as follows.
...
usr/share/man/
usr/share/man/man5/
usr/share/man/man3/
usr/share/man/man1/
usr/share/man/man1/ocamlfind.1.gz
usr/share/man/man5/META.5.gz
usr/share/man/man5/site-lib.5.gz
usr/share/man/man5/findlib.conf.5.gz
...

I've attached a patch to fix the issue in PKGBUILD. Hopefully, 'Makefile-fix.diff' would be submitted upstream so that Arch Linux doesn't need to maintain unneeded patches.
   PKGBUILD.patch (1 KiB)
   Makefile-fix.diff (0.6 KiB)
This task depends upon

Closed by  Alexander F. Rødseth (xyproto)
Monday, 11 July 2016, 17:50 GMT
Reason for closing:  Fixed
Comment by Alif (alive4ever) - Sunday, 12 June 2016, 07:22 GMT
My appologies. The above patch is written after building package.

This one is meant to be applied on clean build.
   PKGBUILD.patch (1 KiB)
   Makefile-fix.diff (0.6 KiB)
Comment by Alif (alive4ever) - Saturday, 18 June 2016, 07:22 GMT
I put patching Makefile on the build() stage. Maybe I should put patching process in the prepare() stage?
Comment by Alif (alive4ever) - Saturday, 18 June 2016, 07:33 GMT
Here is new patches, with patching makefiles placed in prepare() stage.

I hope this will help fixing the packaging issue.
   PKGBUILD.patch (0.9 KiB)
   Makefile-fix.diff (0.6 KiB)
Comment by Alexander F. Rødseth (xyproto) - Wednesday, 06 July 2016, 08:09 GMT
Thanks for the report, I will look into this.

Loading...