RFE: 'port oudated' should provide different output when no outdated ports

[jpm@…]

it would be nice to provide alternate message when there are not any outdated ports.

jpm.giant-squid ~ -=> port version Version: 1.211 jpm.giant-squid ~ -=> port outdated The following installed ports are outdated: jpm.giant-squid ~ -=>

looking at darwinports/base/src/port/port.tcl from cvs, it appears as though it already has this capability. but even after installing from cvs i still get the string "The following installed ports are outdated:" when i do not have any outdated ports.

proc action_outdated { action portlist opts } {

[...] if { [llength $ilist] > 0 } {

puts "The following installed ports are outdated:" [...]

} else {

puts "No installed ports are outdated."

}

return $status

}

[jpm@…]

diff that fixes the errant message being printed

[jpm@…]

okay, i think i know what is happening here - i believe the conditional is checking for something other than what the printed strings is indicating.

$ilist actually contains the list of installed ports (or just the individual ports if the user has specificed them in the outdated command), it does not contain a pruned list indicating the outdated. so that means, unless there aren't any ports installed, then the message

"The following installed ports are outdated:"

will always be printed.

[jpm@…]

i comandeered a machine that had 1.200 installed and tested the patch out and it seems to be working as expected. i'm sure it could use a beautification pass but functionally it seems to be okay.

here is some lovely terminal spew of my quick test:

jpm.freemason ~ -=> port outdated The following installed ports are outdated: jpm.freemason ~ -=> ./port outdated No installed ports are outdated. jpm.freemason ~ -=> sudo port selfupdate DarwinPorts base version 1.200 installed Downloaded DarwinPorts base version 1.211 Configuring, Building and Installing new DarwinPorts base selfupdate done! jpm.freemason ~ -=> port outdated The following installed ports are outdated: apr 1.2.2_0 < 1.2.6_0 apr-util 1.2.2_1 < 1.2.6_0 subversion 1.3.0_2 < 1.3.1_0 jpm.freemason ~ -=> ./port outdated The following installed ports are outdated: apr 1.2.2_0 < 1.2.6_0 apr-util 1.2.2_1 < 1.2.6_0 subversion 1.3.0_2 < 1.3.1_0 jpm.freemason ~ -=> sudo port upgrade outdated [...] jpm.freemason ~ -=> port outdated The following installed ports are outdated: jpm.freemason ~ -=> ./port outdated No installed ports are outdated. jpm.freemason ~ -=>

[jberry@…]

Committed change. Thanks for the patch.