Opened 9 years ago

Closed 9 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 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 Schmidt) 9 years ago.
proposed patch

Download all attachments as: .zip

Change History (8)

comment:1 Changed 9 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 9 years ago by ryandesign (Ryan 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 9 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 9 years ago by ryandesign (Ryan 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 9 years ago by ryandesign (Ryan Schmidt)

Attachment: puts.diff added

proposed patch

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

Cc: jmr@… added

Or what about just the attached?

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

Cc: ryandesign@… added

comment:7 Changed 9 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.