Opened 10 years ago

Last modified 3 years ago

#42193 reopened defect

c2html @0.9.6 tries to skip destroot and install directly into prefix

Reported by: cooljeanius (Eric Gallager) Owned by: macports-tickets@…
Priority: Normal Milestone:
Component: ports Version: 2.2.1
Keywords: Cc: tobypeterson, seanfarley (Sean Farley)
Port: c2html

Description

I am on a new machine on which I can re-enable sandboxing for MacPorts, and while re-installing my ports, the sandbox caught this:

DEBUG: Executing org.macports.destroot (c2html)
DEBUG: Environment: CPATH='/opt/local/include' CC_PRINT_OPTIONS_FILE='/opt/local/var/macports/build/_opt_local_var_macports_sources_rsync.macports.org_release_tarballs_ports_textproc_c2html/c2html/work/.CC_PRINT_OPTIONS' LIBRARY_PATH='/opt/local/lib' CC_PRINT_OPTIONS='YES' MACOSX_DEPLOYMENT_TARGET='10.6'
DEBUG: Assembled command: 'cd "/opt/local/var/macports/build/_opt_local_var_macports_sources_rsync.macports.org_release_tarballs_ports_textproc_c2html/c2html/work/c2html-0.9.6" && /usr/bin/make -w install DESTDIR=/opt/local/var/macports/build/_opt_local_var_macports_sources_rsync.macports.org_release_tarballs_ports_textproc_c2html/c2html/work/destroot'
DEBUG: Executing command line:  cd "/opt/local/var/macports/build/_opt_local_var_macports_sources_rsync.macports.org_release_tarballs_ports_textproc_c2html/c2html/work/c2html-0.9.6" && /usr/bin/make -w install DESTDIR=/opt/local/var/macports/build/_opt_local_var_macports_sources_rsync.macports.org_release_tarballs_ports_textproc_c2html/c2html/work/destroot 
make: Entering directory `/opt/local/var/macports/build/_opt_local_var_macports_sources_rsync.macports.org_release_tarballs_ports_textproc_c2html/c2html/work/c2html-0.9.6'
mkdir -p /opt/local/share/man/man1 /opt/local/var/macports/build/_opt_local_var_macports_sources_rsync.macports.org_release_tarballs_ports_textproc_c2html/c2html/work/destroot/opt/local/share/doc
/usr/bin/install -c -m 644 ./c2html.1 /opt/local/share/man/man1/c2html.1
install: /opt/local/share/man/man1/c2html.1: Operation not permitted
make: *** [install-doc] Error 71
make: Leaving directory `/opt/local/var/macports/build/_opt_local_var_macports_sources_rsync.macports.org_release_tarballs_ports_textproc_c2html/c2html/work/c2html-0.9.6'
Command failed:  cd "/opt/local/var/macports/build/_opt_local_var_macports_sources_rsync.macports.org_release_tarballs_ports_textproc_c2html/c2html/work/c2html-0.9.6" && /usr/bin/make -w install DESTDIR=/opt/local/var/macports/build/_opt_local_var_macports_sources_rsync.macports.org_release_tarballs_ports_textproc_c2html/c2html/work/destroot 
Exit code: 2
Error: org.macports.destroot for port c2html returned: command execution failed
DEBUG: Error code: CHILDSTATUS 22919 2

Change History (10)

comment:1 Changed 10 years ago by cooljeanius (Eric Gallager)

after autoreconf-ing, the configure script adds the following warning which could possibly be related:

config.status: WARNING:  'Makefile.in' seems to ignore the --datarootdir setting

Seeing as the port already patches the Makefile.in here, perhaps it could also add this:

datarootdir = @datarootdir@
Last edited 10 years ago by cooljeanius (Eric Gallager) (previous) (diff)

comment:2 in reply to:  1 ; Changed 10 years ago by seanfarley (Sean Farley)

Replying to egall@…:

after autoreconf-ing, the configure script adds the following warning which could be related:

config.status: WARNING:  'Makefile.in' seems to ignore the --datarootdir setting

Seeing as the port already patches the Makefile.in here, perhaps it could also add this:

datarootdir = @datarootdir@

Since I was the last one to touch this port, I can do this.

comment:3 in reply to:  2 ; Changed 10 years ago by seanfarley (Sean Farley)

Replying to sean@…:

Replying to egall@…:

after autoreconf-ing, the configure script adds the following warning which could be related:

config.status: WARNING:  'Makefile.in' seems to ignore the --datarootdir setting

Seeing as the port already patches the Makefile.in here, perhaps it could also add this:

datarootdir = @datarootdir@

Since I was the last one to touch this port, I can do this.

Fixed in r116154.

comment:4 in reply to:  3 Changed 10 years ago by cooljeanius (Eric Gallager)

Replying to sean@…:

Replying to sean@…:

Replying to egall@…:

after autoreconf-ing, the configure script adds the following warning which could be related:

config.status: WARNING:  'Makefile.in' seems to ignore the --datarootdir setting

Seeing as the port already patches the Makefile.in here, perhaps it could also add this:

datarootdir = @datarootdir@

Since I was the last one to touch this port, I can do this.

Fixed in r116154.

That worked, thanks!

Edit: this ticket can be closed now.

Last edited 10 years ago by cooljeanius (Eric Gallager) (previous) (diff)

comment:5 Changed 10 years ago by jmroot (Joshua Root)

Cc: sean@… added
Resolution: fixed
Status: newclosed

comment:6 Changed 10 years ago by jmroot (Joshua Root)

Resolution: fixed
Status: closedreopened

Wait, the unregistered files still need to be deleted in pre-activate. Grep the ports tree for registry_file_registered for examples.

comment:7 in reply to:  6 ; Changed 10 years ago by cooljeanius (Eric Gallager)

Replying to jmr@…:

Wait, the unregistered files still need to be deleted in pre-activate. Grep the ports tree for registry_file_registered for examples.

Or just look at r116136 as a recent example that I remember...

comment:8 in reply to:  7 ; Changed 10 years ago by seanfarley (Sean Farley)

Replying to egall@…:

Replying to jmr@…:

Wait, the unregistered files still need to be deleted in pre-activate. Grep the ports tree for registry_file_registered for examples.

Or just look at r116136 as a recent example that I remember...

Ah, thanks for the tip. I'll push this soon.

comment:9 in reply to:  8 Changed 10 years ago by seanfarley (Sean Farley)

Replying to sean@…:

Replying to egall@…:

Replying to jmr@…:

Wait, the unregistered files still need to be deleted in pre-activate. Grep the ports tree for registry_file_registered for examples.

Or just look at r116136 as a recent example that I remember...

Ah, thanks for the tip. I'll push this soon.

Hopefully r116177 fixes this.

comment:10 Changed 3 years ago by cooljeanius (Eric Gallager)

Yeah I think this can be closed now

Note: See TracTickets for help on using tickets.