Bug 1266011
Summary: | Review Request: ocaml-re - Pure OCaml regular expressions with support for Perl and POSIX-style strings | ||
---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Christopher Meng <i> |
Component: | Package Review | Assignee: | Nobody's working on this, feel free to take it <nobody> |
Status: | CLOSED DUPLICATE | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
Severity: | medium | Docs Contact: | |
Priority: | medium | ||
Version: | rawhide | CC: | hobbes1069, package-review, rosser.bjr |
Target Milestone: | --- | ||
Target Release: | --- | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Fixed In Version: | Doc Type: | Bug Fix | |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2018-08-23 15:37:37 UTC | Type: | --- |
Regression: | --- | Mount Type: | --- |
Documentation: | --- | CRM: | |
Verified Versions: | Category: | --- | |
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
Cloudforms Team: | --- | Target Upstream Version: | |
Embargoed: |
Description
Christopher Meng
2015-09-24 10:05:26 UTC
Ok, this is my first ocaml review so I have a few questions... 1. Under the OCaml packaging guidelines I see there should be some macros set[1]: %global _use_internal_dependency_generator 0 %global __find_requires /usr/lib/rpm/ocaml-find-requires.sh %global __find_provides /usr/lib/rpm/ocaml-find-provides.sh Per the example spec file, these should go between BuildRequires and %description. Is there I reason your package doesn't need these? In the requires output from fedora-review I don't see the MD5hash as part of the requirement so I'm thinking you do need this. 2. I Don't see a "make opt" or the test at the top of the spec. Does this library not support byte-code only arches? 3. I assume these are not stripped per the OCaml guidelines[2] because they would not function? ocaml-re.x86_64: W: unstripped-binary-or-object /usr/lib64/ocaml/re/re_str.cmxs ocaml-re.x86_64: W: unstripped-binary-or-object /usr/lib64/ocaml/re/re_glob.cmxs ocaml-re.x86_64: W: unstripped-binary-or-object /usr/lib64/ocaml/re/re_pcre.cmxs ocaml-re.x86_64: W: unstripped-binary-or-object /usr/lib64/ocaml/re/re_posix.cmxs ocaml-re.x86_64: W: unstripped-binary-or-object /usr/lib64/ocaml/re/re.cmxs ocaml-re.x86_64: W: unstripped-binary-or-object /usr/lib64/ocaml/re/re_emacs.cmxs ocaml-re.x86_64: W: unstripped-binary-or-object /usr/lib64/ocaml/re/re_perl.cmxs [1] https://fedoraproject.org/wiki/Packaging:OCaml?rd=Packaging/OCaml#Requires_and_provides [2] https://fedoraproject.org/wiki/Packaging:OCaml?rd=Packaging/OCaml#Stripping_binaries jonludlam's scratch build of ocaml-re-1.4.1-1.fc23.src.rpm for f23 completed http://koji.fedoraproject.org/koji/taskinfo?taskID=11536753 *** This bug has been marked as a duplicate of bug 1174036 *** |