[wpkg-users] [Bug 46] New: [patch] Reduce output of /dryrun

bugzilla-daemon at wpkg.org bugzilla-daemon at wpkg.org
Thu Dec 21 22:06:34 CET 2006


Please reply using this URL only: http://bugs.wpkg.org/show_bug.cgi?id=46

           Summary: [patch] Reduce output of /dryrun
           Product: WPKG
           Version: 0.9.10
          Platform: PC
        OS/Version: Windows XP
            Status: NEW
          Severity: enhancement
          Priority: P2
         Component: wpkg.js
        AssignedTo: mangoo at mch.one.pl
        ReportedBy: greg at mtechsolutions.ca
         QAContact: wpkg-users at lists.wpkg.org


Just a patch to reduce the amount of output when /dryrun is used.. combines the
packages, hosts, etc lists into one message box, instead of one for each.

--- wpkg.js.orig        2006-12-21 10:47:20.000000000 -0500
+++ wpkg.js     2006-12-21 15:53:06.175782379 -0500
@@ -288,48 +288,50 @@

        if( debug ) {
                var hst = hosts.selectNodes( "host" );
-               info( "Hosts file contains " + hst.length + " hosts:" );
                var dsds = 0;
+               var infostr = '';
                for( dsds = 0; dsds < hst.length; ++dsds ) {
-                       info( hst[dsds].getAttribute( "name" ) );
+                       infostr += hst[dsds].getAttribute( "name" ) + ' ';
                }
-               info( "" );
+               info( "Hosts file contains " + hst.length + " hosts:\n" + infostr);
+
        }

     if (debug) {
         var packs = settings.selectNodes("package");
-        info("settings file contains " + packs.length + " packages:");
         var dsds=0;
+               var infostr = '';
         for (dsds=0; dsds<packs.length; ++dsds) {
             if (null != packs[dsds]) {
-               info(packs[dsds].getAttribute("id"));
+               infostr += packs[dsds].getAttribute("id") + ' ';
             }
         }
-        info("");
+        info("settings file contains " + packs.length + " packages:\n" + infostr);
     }



I also commented out line 570 (dinfo("testing available package: " +
packageArray[j].getAttribute("id"));), but didn't include that in this patch,
thought it may be useful to do.

-- 
Configure bugmail: http://bugs.wpkg.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.






_______________________________________________
wpkg-users mailing list
wpkg-users at lists.wpkg.org
http://lists.wpkg.org/mailman/listinfo/wpkg-users



More information about the wpkg-users mailing list