Opened 8 years ago

Closed 8 years ago

Last modified 8 years ago

#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)

patch-demeter-Portfile.diff (528 bytes) - added by dbevans (David B. Evans) 8 years ago.
Proposed patch to update build dependencies

Download all attachments as: .zip

Change History (4)

Changed 8 years ago by dbevans (David B. Evans)

Attachment: patch-demeter-Portfile.diff added

Proposed patch to update build dependencies

comment:1 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 in reply to:  1 Changed 8 years ago by dbevans (David B. Evans)

Resolution: fixed
Status: newclosed

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.

Last edited 8 years ago by dbevans (David B. Evans) (previous) (diff)

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.

Note: See TracTickets for help on using tickets.