Opened 7 years ago

Closed 7 years ago

#53352 closed update (fixed)

thrift: update to 0.10.0

Reported by: m4rin (Marin) Owned by: blair (Blair Zajac)
Priority: Normal Milestone:
Component: ports Version: 2.3.5
Keywords: haspatch Cc: Schamschula (Marius Schamschula), kurthindenburg (Kurt Hindenburg), dbevans (David B. Evans), michaelld (Michael Dickens)
Port: thrift p5-thrift

Description

Thrift 0.10.0 is out.

Attempt at a minimally invasive update. Just like the previous version, assumes a multithreaded boost install. Instead of patching the Makefile.in, patches the configure.ac and regenerates the configure file, hence the new dependencies on the autotools chain.

Attachments (1)

Portfile-thrift.diff​ (2.4 KB) - added by m4rin (Marin) 7 years ago.
Updated the Portfile patch with suggested changes

Download all attachments as: .zip

Change History (9)

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

Cc: blair@… removed
Owner: set to blair
Status: new → assigned
Summary: thrift @0.10.0: upgrade thrift to 0.10.0 → thrift: update to 0.10.0

Thanks.

  • The revision should be 0 (just remove the revision line)
  • Running autoreconf and adding its dependencies is usually accomplished by writing use_autoreconf yes

comment:2 Changed 7 years ago by dbevans (David B. Evans)

Cc: Schamschula kurthindenburg added
Port: p5-thrift added

See also #52452. p5-thrift should be updated to match.

comment:3 Changed 7 years ago by dbevans (David B. Evans)

Cc: dbevans added

Changed 7 years ago by m4rin (Marin)

Attachment: Portfile-thrift.diff​ added

Updated the Portfile patch with suggested changes

comment:4 Changed 7 years ago by m4rin (Marin)

Thanks for getting back so quickly. Changed the revision to 0, autoreconf invoked through use_autoreconf yes.

comment:5 Changed 7 years ago by michaelld (Michael Dickens)

Question: Why not just patch the top-level "configure" file to fix Boost library names (instead of "configure.ac")? Same end result, minimally invasive, requires fewer dependencies, and can easily figure out if upstream fixes how Boost is found (because the patch will fail).

comment:6 Changed 7 years ago by michaelld (Michael Dickens)

Cc: michaelld added

comment:7 Changed 7 years ago by michaelld (Michael Dickens)

Odd thing: In my install of boost_unit_test_framework, the .dylib does not contain the symbol "_main" while the .a does. So, I have to link against the latter, not the former. Wondering if this is a feature or a bug?!

comment:8 Changed 7 years ago by blair (Blair Zajac)

Resolution: → fixed
Status: assigned → closed

In 52d3c5a9/macports-ports:

thrift: new upstream 0.10.0 release.

Closes: #53352

Note: See TracTickets for help on using tickets.