New Ticket     Tickets     Wiki     Browse Source     Timeline     Roadmap     Ticket Reports     Search

Ticket #12096 (closed update: fixed)

Opened 6 years ago

Last modified 4 years ago

boost 1.33.1 -> boost 1.34 portfile diff

Reported by: sanchom@… Owned by: ryandesign@…
Priority: Normal Milestone:
Component: ports Version:
Keywords: boost Cc: sanchom@…,ryandesign@…
Port:

Description

I'm submitting a unified diff of the changes to the boost portfile. This brings boost from 1.33.1 to 1.34.

I removed the icu variant because this is automatically detected at the configuration stage.

Attachments

Portfile.diff (4.5 KB) - added by sanchom@… 6 years ago.
boost 1.33.1 -> boost 1.34 portfile upgrade
Portfile.2.diff (739 bytes) - added by sanchom@… 6 years ago.
boost 1.34 portfile diff now includes icu variant

Change History

Changed 6 years ago by sanchom@…

boost 1.33.1 -> boost 1.34 portfile upgrade

comment:1 Changed 6 years ago by nox@…

"I removed the icu variant because this is automatically detected at the configuration stage."

That's actually a very bad thing, if icu is then uninstalled, boost will be broken. icu dependency must be clearly stated in the port.

comment:2 Changed 6 years ago by nox@…

  • Cc n.oxyde@… added

comment:3 Changed 6 years ago by ryandesign@…

  • Owner changed from macports-dev@… to ryandesign@…
  • Cc n.oxyde@…, sanchom@…, ryandesign@… added; n.oxyde@… removed

Committed in r26030; thank you.

comment:4 Changed 6 years ago by ryandesign@…

Oh... sorry... I didn't see the comment by nox before I committed. Yes, I agree, the dependency on icu needs to be declared, as it was in the line you removed:

depends_lib             lib:libicuuc:icu

Note that we now prefer dependencies to be stated using the port: syntax (e.g. "depends_lib port:icu") instead of the lib: syntax.

comment:5 Changed 6 years ago by sanchom@…

Okay, so I reinstated the icu variant. Do you want the diff from the previous 1.33.1 portfile to the current portfile with the icu variant reinstated?

comment:6 Changed 6 years ago by ryandesign@…

Since I've already committed your previous diff, I'd prefer a diff against the portfile from r26030.

Changed 6 years ago by sanchom@…

boost 1.34 portfile diff now includes icu variant

comment:7 Changed 6 years ago by sanchom@…

new diff added

comment:8 Changed 6 years ago by ryandesign@…

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

Portfile.2.diff applied and +icu variant reinstated in r26109. Thanks.

comment:9 Changed 6 years ago by nox@…

  • Cc n.oxyde@… removed
  • Priority changed from Expected to Normal
  • Version 1.4.42 deleted

comment:10 Changed 4 years ago by jmr@…

  • Type changed from enhancement to update

comment:11 Changed 4 years ago by anonymous

  • Milestone Port Updates deleted

Milestone Port Updates deleted

Note: See TracTickets for help on using tickets.