Ticket #12096 (closed update: fixed)
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
Change History
Changed 6 years ago by sanchom@…
- Attachment Portfile.diff added
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: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@…
- Attachment Portfile.2.diff added
boost 1.34 portfile diff now includes icu variant
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:11 Changed 4 years ago by anonymous
- Milestone Port Updates deleted
Milestone Port Updates deleted


boost 1.33.1 -> boost 1.34 portfile upgrade