New Ticket     Tickets     Wiki     Browse Source     Timeline     Roadmap     Ticket Reports     Search

Ticket #30511 (closed defect: fixed)

Opened 22 months ago

Last modified 13 months ago

ngrep fails to build

Reported by: david@… Owned by: grace@…
Priority: Normal Milestone:
Component: ports Version: 2.0.0
Keywords: Cc: arno+macports@…, analogue@…, pipeface@…
Port: ngrep

Description

Log attached.

Attachments

main.log (17.2 KB) - added by david@… 22 months ago.
Portfile.diff (567 bytes) - added by arno+macports@… 21 months ago.
Update to correct MAKECFLAGS prior to configure.

Change History

Changed 22 months ago by david@…

comment:1 Changed 22 months ago by macsforever2000@…

  • Owner changed from macports-tickets@… to grace@…

comment:2 Changed 22 months ago by arno+macports@…

I'm not sure if there's a deeper underlying cause, but it seems to be related to the "make" being performed as "make -j2 -w all". If you drop the "-w" the build proceeds without issue. The flag is being interpreted somewhere as a build target named "w" instead of as an argument to make.

At least in my case, I also had to disable the universal build. i386 wasn't happy, but x86_64 was.

comment:3 Changed 22 months ago by arno+macports@…

  • Cc arno+macports@… added

Cc Me!

comment:4 Changed 22 months ago by analogue@…

  • Cc analogue@… added

Cc Me!

comment:5 Changed 22 months ago by arno+macports@…

OK, this is a bug in the ngrep Makefile.in.

As described in [this http://www.mail-archive.com/bug-make@gnu.org/msg00102.html thread], MAKEFLAGS should not be passed recursively to calls to 'make' within the Makefile. Line 66 of Makefile.in contains such an error. Removing the "$(MAKEFLAGS)" bit from that line allows ngrep to build with "make -j2 -w all".

A reinplace should be enough to strip that out prior to calling configure.

I think the following should handle it.

pre-configure {
    reinplace s|\$(MAKE) \$(MAKEFLAGS)|$(MAKE)| ${worksrcpath}/Makefile.in
}

comment:6 Changed 22 months ago by pipeface@…

  • Cc pipeface@… added

Cc Me!

Changed 21 months ago by arno+macports@…

Update to correct MAKECFLAGS prior to configure.

comment:7 Changed 21 months ago by arno+macports@…

I've attached a patch which updates the Portfile to correct the MAKECFLAGS issue. It also disables universal building as that seems to be broken. And bumps the revision for good measure (I'm not sure if this is necessary or not).

comment:8 Changed 21 months ago by arno+macports@…

This is corrected by changeset r83135.

It's still failing to build +universal for me under Lion though. I'll open a ticket later today.

comment:9 Changed 13 months ago by jmr@…

  • Status changed from new to closed
  • Resolution set to fixed

Universal fixed in r92800.

Note: See TracTickets for help on using tickets.