Opened 14 years ago

Closed 14 years ago

#22740 closed defect (fixed)

pthsem: configure is not longer available

Reported by: mail@… Owned by: nerdling (Jeremy Lavergne)
Priority: Normal Milestone:
Component: ports Version:
Keywords: maintainer haspatch Cc: roederja
Port: pthsem

Description

the build has to use autoconf/automake instead (at least as long as a git checkout has to be used instead of a tarball)

Attachments (2)

Portfile.diff (1.3 KB) - added by mail@… 14 years ago.
pthsem.diff (1.2 KB) - added by nerdling (Jeremy Lavergne) 14 years ago.
successfully installed 2.0.8

Download all attachments as: .zip

Change History (15)

comment:1 Changed 14 years ago by ryandesign (Ryan Carsten Schmidt)

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

If this port used to work, then it sounds like a problem is that the port does not always install the same software -- it installs whatever's at the head of the git repository. The port should instead assure that always the same version of software is used, for example by using the git.branch keyword.

But this brings up a point: why is the port using git and not the tarball available on the project's web site?

Changed 14 years ago by mail@…

Attachment: Portfile.diff added

comment:2 Changed 14 years ago by mail@…

  1. the port is using git instead of the tarball, because the changes for Mac OS X are ot available in the tarball yet. I've talked to the developer, he will release a new version at mid 2010, after that i will retarget the port to the tarball.
  1. The portfile has now a git.branch, which points to the (at this moment) head.

Thanks for looking into it ;)

comment:3 Changed 14 years ago by roederja

Cc: jann@… added

Cc Me!

comment:4 Changed 14 years ago by nerdling (Jeremy Lavergne)

I'd suggest using use_autoreconf yes in place of all the manual commands. Once it builds successfully for me I'll commit it.

comment:5 Changed 14 years ago by nerdling (Jeremy Lavergne)

Also, if I'm not mistaken, 2.0.8 is available?

Changed 14 years ago by nerdling (Jeremy Lavergne)

Attachment: pthsem.diff added

successfully installed 2.0.8

comment:6 Changed 14 years ago by nerdling (Jeremy Lavergne)

I've attached my diff of the portfile for 2.0.8. Let me know if it's okay and I'll commit it.

comment:7 Changed 14 years ago by ryandesign (Ryan Carsten Schmidt)

Owner: changed from ryandesign@… to snc@…
Status: assignednew

comment:8 Changed 14 years ago by mail@…

now configure is used (instead of autoconf/automake) and no git checkout, should we ommit

depends_build

port:autoconf \ port:automake \ port:git-core

comment:9 in reply to:  8 Changed 14 years ago by nerdling (Jeremy Lavergne)

Replying to mail@…:

Yes, good point. I missed the deps as I was removing the macro instead.

comment:10 Changed 14 years ago by nerdling (Jeremy Lavergne)

Is everything okay with the patch I posted (in addition to axing the deps listed above?

comment:11 in reply to:  10 Changed 14 years ago by mail@…

Yep, everything works fine for me on 10.5 and 10.6 :)

comment:12 Changed 14 years ago by nerdling (Jeremy Lavergne)

Status: newassigned

comment:13 Changed 14 years ago by nerdling (Jeremy Lavergne)

Resolution: fixed
Status: assignedclosed

Committed in r64227.

Note: See TracTickets for help on using tickets.