[Bug 1862625] Review Request: ocaml-ppx-js-style - Code style checker for Jane Street OCaml packages

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



https://bugzilla.redhat.com/show_bug.cgi?id=1862625



--- Comment #3 from Jerry James <loganjerry@xxxxxxxxx> ---
(In reply to Fabio Valentini from comment #2)
> The only thing I'm curious about is the 17.4 MB (?) ppx.exe file that's
> included in the main package.
> What is it? .exe files are not mentioned in the OCaml Packaging Guidelines.
> It also looks unreasonably large compared to the size of the source files.

Thank you for asking that question.  It motivated me to finally figure out how
to tell dune to show command lines as it builds projects.  Now I have another
change to make to a bunch of OCaml packages.  (For the record, "dune
--display=verbose" does the trick.)

This is the step that creates ppx.exe:

(cd _build/default && /usr/bin/ocamlopt.opt -g -o
.ppx/0224ad3443a846e54f1637fccb074e7d/ppx.exe -w -24 -I /usr/lib64/ocaml/base
-I /usr/lib64/ocaml/base/base_internalhash_types -I /usr/lib64/ocaml/base/caml
-I /usr/lib64/ocaml/base/shadow_stdlib -I /usr/lib64/ocaml/compiler-libs -I
/usr/lib64/ocaml/ocaml-compiler-libs/common -I
/usr/lib64/ocaml/ocaml-compiler-libs/shadow -I
/usr/lib64/ocaml/ocaml-migrate-parsetree -I /usr/lib64/ocaml/ppx_derivers -I
/usr/lib64/ocaml/ppxlib -I /usr/lib64/ocaml/ppxlib/ast -I
/usr/lib64/ocaml/ppxlib/metaquot -I /usr/lib64/ocaml/ppxlib/metaquot_lifters -I
/usr/lib64/ocaml/ppxlib/print_diff -I /usr/lib64/ocaml/ppxlib/traverse_builtins
-I /usr/lib64/ocaml/result -I /usr/lib64/ocaml/sexplib0 -I
/usr/lib64/ocaml/stdio /usr/lib64/ocaml/compiler-libs/ocamlcommon.cmxa
/usr/lib64/ocaml/ocaml-compiler-libs/common/ocaml_common.cmxa
/usr/lib64/ocaml/ocaml-compiler-libs/shadow/ocaml_shadow.cmxa
/usr/lib64/ocaml/result/result.cmxa
/usr/lib64/ocaml/ppx_derivers/ppx_derivers.cmxa
/usr/lib64/ocaml/ocaml-migrate-parsetree/migrate_parsetree.cmxa
/usr/lib64/ocaml/ppxlib/ast/ppxlib_ast.cmxa
/usr/lib64/ocaml/base/base_internalhash_types/base_internalhash_types.cmxa
/usr/lib64/ocaml/base/caml/caml.cmxa /usr/lib64/ocaml/sexplib0/sexplib0.cmxa
/usr/lib64/ocaml/base/shadow_stdlib/shadow_stdlib.cmxa
/usr/lib64/ocaml/base/base.cmxa /usr/lib64/ocaml/stdio/stdio.cmxa
/usr/lib64/ocaml/ppxlib/print_diff/ppxlib_print_diff.cmxa
/usr/lib64/ocaml/ppxlib/traverse_builtins/ppxlib_traverse_builtins.cmxa
/usr/lib64/ocaml/ppxlib/ppxlib.cmxa
/usr/lib64/ocaml/ppxlib/metaquot_lifters/ppxlib_metaquot_lifters.cmxa
/usr/lib64/ocaml/ppxlib/metaquot/ppxlib_metaquot.cmxa
.ppx/0224ad3443a846e54f1637fccb074e7d/_ppx.ml)

So that executable is built out of a number of static libraries (cmxa) pulled
from other OCaml packages, which is why it is so large.

New URLs with verbose builds and 'with' instead of 'and':
Spec URL:
https://jjames.fedorapeople.org/ocaml-ppx-js-style/ocaml-ppx-js-style.spec
SRPM URL:
https://jjames.fedorapeople.org/ocaml-ppx-js-style/ocaml-ppx-js-style-0.14.0-1.fc33.src.rpm


-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
_______________________________________________
package-review mailing list -- package-review@xxxxxxxxxxxxxxxxxxxxxxx
To unsubscribe send an email to package-review-leave@xxxxxxxxxxxxxxxxxxxxxxx
Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: https://lists.fedoraproject.org/archives/list/package-review@xxxxxxxxxxxxxxxxxxxxxxx




[Index of Archives]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite Conditions]     [KDE Users]

  Powered by Linux