Opened 11 years ago

Closed 11 years ago

Last modified 11 years ago

#38412 closed defect (fixed)

rpm, rpm45, rpm50, rpm51, rpm52, rpm53, rpm54: building with +docs variant fails if doxygen is not active

Reported by: ryandesign (Ryan Carsten Schmidt) Owned by: n3npq@…
Priority: Normal Milestone:
Component: ports Version: 2.1.3
Keywords: haspatch Cc: afb@…, cooljeanius (Eric Gallager)
Port: rpm, rpm45, rpm50, rpm51, rpm52, rpm53, rpm54

Description

$ sudo port deact doxygen
--->  Deactivating doxygen @1.8.3.1_0+universal
$ sudo port build rpm54 +docs -universal
--->  Computing dependencies for rpm54
--->  Fetching distfiles for rpm54
--->  Verifying checksum(s) for rpm54
--->  Extracting rpm54
--->  Configuring rpm54
Error: org.macports.configure for port rpm54 returned: configure failure: command execution failed
Please see the log file for port rpm54 for details:
    /opt/local/var/macports/logs/_Users_rschmidt_macports_dports_sysutils_rpm54/rpm54/main.log
To report a bug, follow the instructions in the guide:
    http://guide.macports.org/#project.tickets
Error: Processing of port rpm54 failed

Log contains:

checking for doxygen... no
configure: error: RPM API docs need doxygen in PATH

doxygen is typically a build dependency, not a runtime dependency.

Also it should be a port:-style dependency, not a bin:-style one. We don't want possibly old and/or broken doxygens the user might have installed elsewhere to satisfy the dependency; we want to use the MacPorts one that we more or less know will work.

Attachments (1)

rpm.diff (3.1 KB) - added by ryandesign (Ryan Carsten Schmidt) 11 years ago.

Download all attachments as: .zip

Change History (3)

Changed 11 years ago by ryandesign (Ryan Carsten Schmidt)

Attachment: rpm.diff added

comment:1 Changed 11 years ago by afb@…

Resolution: fixed
Status: newclosed

Applied, r104136.

Last edited 11 years ago by afb@… (previous) (diff)

comment:2 Changed 11 years ago by cooljeanius (Eric Gallager)

Cc: egall@… added

Cc Me!

Note: See TracTickets for help on using tickets.