Opened 13 years ago

Closed 13 years ago

#30347 closed enhancement (fixed)

Provide progress feedback during images_to_archives.tcl processing

Reported by: ejtttje@… Owned by: macports-tickets@…
Priority: Normal Milestone: MacPorts 2.0.1
Component: base Version: 2.0.0
Keywords: Cc: jmroot (Joshua Root), ryandesign (Ryan Carsten Schmidt)
Port:

Description

When processing the images_to_archives.tcl script, it reports: "This could take a while..."

True enough, the problematic part is that it uses only minimal CPU during this time, which may lead users to believe something has gotten hung up and attempt some kind of 'corrective' measure. I have to admit, I was *just* about to stop the process and re-run this script manually when it finally kicked out literally at the last second.

I'm not sure what this process is doing, perhaps it is doing something disk-heavy and hence very little CPU is involved?

My suggestion is to provide some kind of ongoing progress indicator, beyond the 'take a while' warning, to assure users the upgrade hasn't stalled. Thanks!

Attachments (1)

puts.diff (508 bytes) - added by ryandesign (Ryan Carsten Schmidt) 13 years ago.
proposed patch

Download all attachments as: .zip

Change History (8)

comment:1 Changed 13 years ago by jmroot (Joshua Root)

Summary: Slow upgrade 2.0.0: images_to_archives.tcl processing?Provide progress feedback during images_to_archives.tcl processing

comment:2 Changed 13 years ago by ryandesign (Ryan Carsten Schmidt)

Yes, it is rearranging things on disk. This is a one-time operation for users upgrading from MacPorts 1.x to 2.0.0.

It might be a little late to think about adding a progress indicator now; by the time someone wrote and tested that code and we got around to releasing another version of MacPorts, everyone will already have gone through the upgrade process.

comment:3 Changed 13 years ago by ejtttje@…

Fair enough, the secondary hope is that this will get indexed so if people search for the warning text they'll find this ticket to reassure them to just give it time :)

comment:4 Changed 13 years ago by ryandesign (Ryan Carsten Schmidt)

Milestone: MacPorts 2.0.1

Could we maybe just print a "." for every port updated? That would already help a great deal. On my G4 the conversion took over 2 hours. And if we do release 2.0.1 very soon, this might still help a number of people.

Changed 13 years ago by ryandesign (Ryan Carsten Schmidt)

Attachment: puts.diff added

proposed patch

comment:5 Changed 13 years ago by ryandesign (Ryan Carsten Schmidt)

Cc: jmr@… added

Or what about just the attached?

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

Cc: ryandesign@… added

comment:7 Changed 13 years ago by jmroot (Joshua Root)

Resolution: fixed
Status: newclosed

Committed a bit more fancy version in r81464.

Note: See TracTickets for help on using tickets.