#20489 closed defect (fixed)
devel/caml-json-wheel installs file outside of destroot
Reported by: | anil@… | Owned by: | nerdling (Jeremy Lavergne) |
---|---|---|---|
Priority: | Normal | Milestone: | |
Component: | ports | Version: | 1.7.1 |
Keywords: | Cc: | ryandesign (Ryan Carsten Schmidt) | |
Port: | caml-json-wheel |
Description
This small patch fixes it:
Index: Portfile =================================================================== --- Portfile (revision 54590) +++ Portfile (working copy) @@ -26,6 +26,7 @@ port:caml-ocamlnet use_configure no +destroot.env-append BINDIR=${destroot}/${prefix}/bin post-patch { set ocaml_site_path [exec ocamlfind printconf destdir]
Change History (4)
comment:1 Changed 15 years ago by nerdling (Jeremy Lavergne)
Owner: | changed from macports-tickets@… to snc@… |
---|---|
Port: | caml-json-wheel added |
Status: | new → assigned |
comment:2 Changed 15 years ago by nerdling (Jeremy Lavergne)
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
comment:3 Changed 15 years ago by ryandesign (Ryan Carsten Schmidt)
Cc: | ryandesign@… added |
---|
Jeremy, I corrected the log message from r54632. This wasn't an mtree violation; it was that the port installed a file directly into ${prefix}/bin, outside of ${destroot}.
Anil, a few notes:
- Since this changes what files are registered to the port, you must increase the port's revision (if you are not already increasing the version)
- There's no need to put a slash before ${prefix} because ${prefix} already begins with a slash
- Using code I copied from my lisaem port, I added a pre-activate phase to remove the rogue file from ${prefix}/bin if it's there; otherwise, the user would get this message:
---> Activating caml-json-wheel @1.0.6_1 Error: Target org.macports.activate returned: Image error: /opt/local/bin/jsoncat already exists and does not belong to a registered port. Unable to activate port caml-json-wheel. Error: Status 1 encountered during processing.
I committed these changes in r54686.
Note: See
TracTickets for help on using
tickets.
Committed in r54632.