Opened 9 years ago

Closed 9 years ago

#48892 closed enhancement (fixed)

httping 2.4 no longer has ncurses mode

Reported by: breiter (Brian Reiter) Owned by: ryandesign (Ryan Carsten Schmidt)
Priority: Normal Milestone:
Component: ports Version:
Keywords: haspatch Cc:
Port: httping

Description

httping 2.4 is linked against ncurses but it doesn't have a -K option in the resulting binary anymore. I think what has to happen is uncommenting NC=yes and FW=yes in the Makefile or adding those lines to makefile.inc.

Also may want to add TFO=yes.

The configure shell script will set these options. I've attached a patch that enables ncurses mode.

Attachments (1)

Portfile.diff (516 bytes) - added by breiter (Brian Reiter) 9 years ago.
Portfile diff that adds configure script invocation to fix ncurses

Download all attachments as: .zip

Change History (4)

Changed 9 years ago by breiter (Brian Reiter)

Attachment: Portfile.diff added

Portfile diff that adds configure script invocation to fix ncurses

comment:1 Changed 9 years ago by mf2k (Frank Schima)

Keywords: haspatch added
Type: defectenhancement
Version: 2.3.3

comment:2 Changed 9 years ago by ryandesign (Ryan Carsten Schmidt)

Owner: changed from macports-tickets@… to ryandesign@…
Status: newassigned

Thanks.

To use a configure script, we would just remove "use_configure no". We would not add "use_configure yes", because that is the default.

The default value of configure.cmd is "./configure", which is what we would use here. We would add any needed arguments to configure.args. By default MacPorts will also put "--prefix=${prefix}" into configure.pre_args; if that is unsuitable for this configure script, it can be cleared.

This configure script uses mktemp in a way that is not compatible with OS X, which causes error messages to be printed and probably causes the temp files not to be created, which could cause the configure script to function incorrectly. I see you're attempting to redirect stderr to /dev/null, perhaps to hide these errors. I'd prefer to fix the configure script so that is uses mktemp correctly.

Your changes introduced tab characters into the portfile, but you should try to match the existing whitespace of the portfile, which in this case uses spaces. That's our preferred style, and it's also what the modeline of this portfile states.

Probably, the extra library flags being added to LDFLAGS will no longer be needed once the configure script is being used.

comment:3 Changed 9 years ago by ryandesign (Ryan Carsten Schmidt)

Resolution: fixed
Status: assignedclosed

r140334. In addition to fixing the configure script's usage of mktemp, I made it remember the values of CC, CFLAGS and LDFLAGS, so that most of the custom code in the portfile isn't needed anymore.

Note: See TracTickets for help on using tickets.