Opened 16 years ago

Closed 16 years ago

Last modified 16 years ago

#17128 closed enhancement (fixed)

RFE: change faad2 depends_lib to depends_build

Reported by: MarcusCalhoun-Lopez (Marcus Calhoun-Lopez) Owned by: pguyot (Paul Guyot)
Priority: Normal Milestone:
Component: ports Version: 1.6.0
Keywords: Cc:
Port: faad2

Description

I believe that faad2 only depends on libtool, autoconf, and automake
during the build stage.

I don't think it depends on perl at all.

Attached is the proposed change.

Attachments (1)

Portfile.diff (417 bytes) - added by MarcusCalhoun-Lopez (Marcus Calhoun-Lopez) 16 years ago.

Download all attachments as: .zip

Change History (6)

comment:1 Changed 16 years ago by pguyot (Paul Guyot)

Have you tried with -t to back your assumption?

I probably cannot get more spam, and I'd rather have my e-mail the same on all ports so I can easily grep.

Changed 16 years ago by MarcusCalhoun-Lopez (Marcus Calhoun-Lopez)

Attachment: Portfile.diff added

comment:2 Changed 16 years ago by MarcusCalhoun-Lopez (Marcus Calhoun-Lopez)

Trying to install in trace mode results in an error:

aclocal: aclocal: file `/opt/local/share/aclocal/wxwin.m4' does not exist

The offending perl code in aclocal is:

fatal "$where: file `$file' does not exist" if ! -e $file;

There is probably no easy fix to this.

I could find no evidence that libtool, autoconf, or automake were needed after building.
Are they ever?
I could find no mention of perl anywhere.

These dependencies where initially introduced in r19951.

I modified the patch file to keep the old address.

comment:3 Changed 16 years ago by MarcusCalhoun-Lopez (Marcus Calhoun-Lopez)

May I go ahead and commit this patch?

comment:4 Changed 16 years ago by MarcusCalhoun-Lopez (Marcus Calhoun-Lopez)

Resolution: fixed
Status: newclosed

Fixed in r42217.

comment:5 Changed 16 years ago by (none)

Milestone: Port Enhancements

Milestone Port Enhancements deleted

Note: See TracTickets for help on using tickets.