<html><head><meta http-equiv="Content-Type" content="text/html charset=windows-1252"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;">Hi Kyle,<div><br></div><div><br></div><div>Thanks for working on this! I have some comments below:</div><div><br><div><div><div>On Aug 29, 2014, at 12:39 PM, Kyle Sammons <<a href="mailto:ksammons@macports.org">ksammons@macports.org</a>> wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div dir="ltr"><div>port reclaim - Removes all unneeded distfiles currently on the system, and offers to uninstall all inactive applications, if the user desires. </div></div></blockquote><div><br></div><div>I tried to use this for the first time today. Here are my observations:</div><div><br></div><div>1. First, it found an inactive port, but the default action is not clear. </div><div><br></div><div><font face="Monaco">$ sudo port reclaim<br>Found inactive ports: telepathy-glib.<br>Would you like to uninstall these apps? [Y/N]: </font></div><div><br></div><div>This is good but it is not clear what the default action is if you hit return. It is apparently “N”, so the “Y” should be lower case. i.e. <span style="font-family: Monaco;">[y/N]</span></div><div><br></div><div>2. The default output is *way* too verbose. </div><div><font face="Monaco">Found distfile: ipython-2.1.0.tar.gz<br>Removing distfile: ipython-2.1.0.tar.gz<br>Found distfile: LightAquaBlue-framework-MacOS10.6.zip<br>Removing distfile: LightAquaBlue-framework-MacOS10.6.zip<br>Found distfile: LightAquaBlue-python-MacOS10.6.zip<br>Removing distfile: LightAquaBlue-python-MacOS10.6.zip<br>Found distfile: lightblue-0.4.tar.gz<br>Removing distfile: lightblue-0.4.tar.gz<br>Found empty directory: /opt/local/var/macports/distfiles/py-lightblue. Attempting to delete.<br>Found distfile: qutip-3.0.1.tar.gz<br>…</font></div><div><br></div><div>I think it should mention what it is doing in general at first because it will sit there working and then summarize in a manner such as "deleted X distfiles and Y empty directories” and maybe say how much space was saved?</div><div><br></div><div>The current output would be fine if using -v or -d. </div><div><br></div><div>3. It hit an error on mine:</div><div><font face="Monaco">...</font></div><div><font face="Monaco">Found empty directory: /Users/schimaf/.cpan/build/Moose-2.1210-ZIZYEB/bin. Attempting to delete.<br>Found distfile: Changes<br>Removing distfile: Changes<br>Found distfile: Changes.Class-MOP<br>Removing distfile: Changes.Class-MOP<br>Found distfile: dist.ini<br>Removing distfile: dist.ini<br>Found distfile: moosex-compile<br>Removing distfile: moosex-compile<br>too many nested evaluations (infinite loop?)<br> (procedure "ui_message" line 2)<br> invoked from within<br>"ui_msg "Found distfile: $item""<br> (procedure "walk_files" line 45)<br> invoked from within<br>"walk_files $currentPath $delete $dist_paths"<br> (procedure "walk_files" line 37)<br> invoked from within<br>"walk_files $up_dir $delete $dist_paths"<br> (procedure "walk_files" line 27)<br> invoked from within<br>"walk_files $dir $delete $dist_paths"<br> (procedure "walk_files" line 60)<br> invoked from within<br>"walk_files $currentPath $delete $dist_paths"<br> (procedure "walk_files" line 37)<br> invoked from within<br>"walk_files $up_dir $delete $dist_paths"<br> (procedure "walk_files" line 27)<br> invoked from within<br>"walk_files $dir $delete $dist_paths"<br> (procedure "walk_files" line 60)<br> invoked from within<br>"walk_files $currentPath $delete $dist_paths"<br> (procedure "walk_files" line 37)<br>...</font></div><div><br></div><blockquote type="cite"><div dir="ltr"><div>port doctor - Checks for common issues on the users system, and if found, suggests ways to fix the problem. Doctor accepts one flag (--quiet) which only shows warnings/errors rather than showing the status of each check. There is also a configuration file for doctor located in, "/opt/local/var/macports/port_doctor.ini", which allows the user to make changes to certain ways doctor operates, though currently the config file isn't used as much as it could be. </div></div></blockquote><div><br></div><div>I like “diagnose” out of the suggestions offered. </div><div><br></div><br></div><div>Cheers!</div><div>Frank</div><br></div></div></body></html>