#51811 closed defect (fixed)
demeter @0.9.24: change of dependencies in DemeterBuild
Reported by: | dbevans (David B. Evans) | Owned by: | joefowler (Joe Fowler) |
---|---|---|---|
Priority: | Normal | Milestone: | |
Component: | ports | Version: | 2.3.4 |
Keywords: | haspatch | Cc: | kurthindenburg (Kurt Hindenburg) |
Port: | demeter |
Description
On updating demeter to version 0.9.24, I noticed the following dependency error during configure:
Checking prerequisites... build_requires: ! File::Slurper is not installed ERRORS/WARNINGS FOUND IN PREREQUISITES. You may wish to install the versions of the modules indicated above before proceeding with this installation
Indeed, Changes.org shows that as of the 0.9.23 update, DemeterBuild now uses File::Slurper instead of File::Slurp::Tiny. The rationale for this change is outlined here.
Attached is a trivial patch to make this change to demeter's build dependencies. Since the patch only effects the build phase and does not seem to effect the build or test results, there's no need for a revbump.
Let me know if you agree to this change and I'll be glad to commit it for you.
Attachments (1)
Change History (4)
Changed 8 years ago by dbevans (David B. Evans)
Attachment: | patch-demeter-Portfile.diff added |
---|
comment:1 follow-up: 2 Changed 8 years ago by kurthindenburg (Kurt Hindenburg)
Thanks for noticing this. It seems trace didn't catch this. Please go ahead and commit or tell me if you want me to.
By the way, how did you catch this?
comment:2 Changed 8 years ago by dbevans (David B. Evans)
Resolution: | → fixed |
---|---|
Status: | new → closed |
Replying to khindenburg@…:
By the way, how did you catch this?
Pretty much as I explained in the ticket description:
$ sudo port -d configure
which produced the (non-fatal) dependency error message above. Nothing fancy. The Changes.org file explained the rest.
Patch committed in r150039.
comment:3 Changed 8 years ago by joefowler (Joe Fowler)
devans@ and khindenburg@
Thanks very much for the help and for catching the non-fatal error that I missed when I submitted #51809 on Friday (the demeter package rev bump.
Proposed patch to update build dependencies