Opened 12 years ago

Closed 12 years ago

Last modified 12 years ago

#35003 closed update (fixed)

pv @1.2.0 request to update port to the latest version

Reported by: nonstop.server@… Owned by: ryandesign (Ryan Carsten Schmidt)
Priority: Normal Milestone:
Component: ports Version: 2.1.1
Keywords: haspatch Cc: meissnem@…
Port: pv

Description

Request to update port pv to the latest release (currently version 1.3.1).
A difference report of the Portfile is attached.
Note: The existing patch stat64.diff is successfully applied with fuzz 1 for file src/pv/loop.c, if this is regarded as an error or unwanted a new patch file should be created.

Attachments (3)

Portfile-pv.diff (2.9 KB) - added by nonstop.server@… 12 years ago.
proposed patch
Portfile-pv_whitespace.diff (2.7 KB) - added by nonstop.server@… 12 years ago.
proposed patch for whitespace changes
Portfile-pv_functional.diff (1.3 KB) - added by nonstop.server@… 12 years ago.
proposed patch for functional changes

Download all attachments as: .zip

Change History (11)

Changed 12 years ago by nonstop.server@…

Attachment: Portfile-pv.diff added

proposed patch

comment:1 Changed 12 years ago by meissnem@…

nonstop.server@ -- are you interested in taking on the maintainer role? I rarely use pv anymore, so I'd rather it be in the hands of someone who cares...

comment:2 in reply to:  1 Changed 12 years ago by nonstop.server@…

Replying to meissnem@…:

Hi meissnem,
Thank you for your proposition however I'm not sure how much time I'll be able to spend on macports in the long run, so I would like to keep it the way it is.
I don't mind providing patches where and when I can without actually being the official maintainer.

comment:3 Changed 12 years ago by ryandesign (Ryan Carsten Schmidt)

The attached diff contains both whitespace and functional changes. Please supply diffs that do only one or the other.

Changed 12 years ago by nonstop.server@…

Attachment: Portfile-pv_whitespace.diff added

proposed patch for whitespace changes

Changed 12 years ago by nonstop.server@…

Attachment: Portfile-pv_functional.diff added

proposed patch for functional changes

comment:4 in reply to:  3 Changed 12 years ago by nonstop.server@…

Replying to ryandesign@…:

The original diff has been split up in two diffs for whitespace and functional changes respectively.
Please see the newly attached difference reports.

comment:5 Changed 12 years ago by meissnem@…

These new patches look good to me. I do not have commit privileges, so ryandesign@ if you're still listening and would be willing to commit it that would be great.

comment:6 Changed 12 years ago by ryandesign (Ryan Carsten Schmidt)

Owner: changed from macports-tickets@… to ryandesign@…
Status: newassigned

Thank you. But note that if we're going to make whitespace changes, then we want the port's fixed whitespace to conform to our style guidelines, which are 4 spaces per indent; that's what the modeline you're adding says too. I've committed the corrected whitespace and formatting changes in r95761.

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

Resolution: fixed
Status: assignedclosed

As for the functional changes, note that we want to use the rmd160 / sha256 checksum pair these days. There don't appear to be any downloads for this at Google Code (anymore?) so I removed that address from the master_sites. 1.3.4 is available in the mean time. I refreshed the stat64 patch. I committed the functional changes in r95762.

comment:8 in reply to:  7 Changed 12 years ago by nonstop.server@…

Replying to ryandesign@…:

Thank you for the upgrade and all the corrections.
I'll leave out the md5 checksum next time and keep white-spaces at four spaces in future patches.

Note: See TracTickets for help on using tickets.