Opened 20 years ago

Closed 20 years ago

Last modified 19 years ago

#2451 closed defect (fixed)

portchecksum.tcl not reporting correct checksum

Reported by: jberry@… Owned by: pguyot (Paul Guyot)
Priority: Normal Milestone:
Component: base Version: 1.0
Keywords: Cc:
Port:

Description

With Paul's revision to support sha1, we seem to have lost the ability for the code to report at verbose level what the correct checksum should have been. That was a useful feature for building Portfiles and/ or updating correct checksums.

I revised the code as follows: (patch is attached)

  • Refactor checksum calculators check_md5 and check_sha1 as calc_md5 and calc_sha1. They now

simply calculate the appropriate type of checksum and return it. The calling code does the actual checksum verification and thus has the "correct" checksum to report at verbose level.

  • Don't bail from checksum checking loop on first error. Thus if there are multiple checksums, all

checksum errors will be reported. This makes it easier/faster to set multiple checksums at once.

Attachments (2)

patch-portchecksum.tcl (2.3 KB) - added by jberry@… 20 years ago.
patch to base/src/port1.0/portchecksum.tcl
patch-portchecksum.2.tcl (2.3 KB) - added by jberry@… 20 years ago.
patch to base/src/port1.0/portchecksum.tcl

Download all attachments as: .zip

Change History (4)

Changed 20 years ago by jberry@…

Attachment: patch-portchecksum.tcl added

patch to base/src/port1.0/portchecksum.tcl

Changed 20 years ago by jberry@…

Attachment: patch-portchecksum.2.tcl added

patch to base/src/port1.0/portchecksum.tcl

comment:1 Changed 20 years ago by jberry@…

attachments.isobsolete: 01

comment:2 Changed 20 years ago by pguyot (Paul Guyot)

Resolution: fixed
Status: newclosed

Committed. Thanks James.

Note: See TracTickets for help on using tickets.