[paludis-commits] r4585 - in trunk: . src/clients/reconcilio
dleverton at svn.pioto.org
dleverton at svn.pioto.org
Sun Apr 20 01:21:36 UTC 2008
Author: dleverton
Date: 2008-04-20 01:21:35 +0000 (Sun, 20 Apr 2008)
New Revision: 4585
Modified:
trunk/ChangeLog
trunk/NEWS
trunk/src/clients/reconcilio/command_line.cc
trunk/src/clients/reconcilio/command_line.hh
trunk/src/clients/reconcilio/fix_linkage.cc
Log:
Remove --verbose option from reconcilio, and turn on the behaviour unconditionally.
Modified: trunk/ChangeLog
===================================================================
--- trunk/ChangeLog 2008-04-20 00:25:22 UTC (rev 4584)
+++ trunk/ChangeLog 2008-04-20 01:21:35 UTC (rev 4585)
@@ -5,6 +5,11 @@
only listed in SVN log. For a summary of what has changed between releases,
see the NEWS file. This file is occasionally pruned to ChangeLog.old.bz2.
+2008-04-20 David Leverton
+
+ * src/clients/reconcilio/: Remove --verbose option from
+ reconcilio, and turn on the behaviour unconditionally.
+
2008-04-19 David Leverton
* paludis/util/, src/output/: Make Ctrl-C behave better.
Modified: trunk/NEWS
===================================================================
--- trunk/NEWS 2008-04-20 00:25:22 UTC (rev 4584)
+++ trunk/NEWS 2008-04-20 01:21:35 UTC (rev 4585)
@@ -4,6 +4,10 @@
This file lists the major changes between versions. For a more detailed list
of every change, see the ChangeLog.
+trunk/:
+ * reconcilio no longer accepts the --verbose switch; verbose display is
+ now turned on unconditionally.
+
0.26.0_pre2:
* paludis now supports ${root} in environment.conf. This can be
used when defining the "world" key, so that the configuration
Modified: trunk/src/clients/reconcilio/command_line.cc
===================================================================
--- trunk/src/clients/reconcilio/command_line.cc 2008-04-20 00:25:22 UTC (rev 4584)
+++ trunk/src/clients/reconcilio/command_line.cc 2008-04-20 01:21:35 UTC (rev 4585)
@@ -42,7 +42,6 @@
a_no_color(&a_no_colour, "no-color"),
a_environment(&general_args, "environment", 'E', "Environment specification (class:suffix, both parts optional)"),
a_exact(&general_args, "exact", '\0', "Rebuild the same package version that is currently installed", true),
- a_verbose(&general_args, "verbose", 'v', "Display more detailed output", true),
a_resume_command_template(&general_args, "resume-command-template", '\0',
"Save the resume command to a file. If the filename contains 'XXXXXX', use mkstemp(3) to generate the filename"),
a_compact(&general_args, "compact", '\0', "Display output using one line per entry", true),
Modified: trunk/src/clients/reconcilio/command_line.hh
===================================================================
--- trunk/src/clients/reconcilio/command_line.hh 2008-04-20 00:25:22 UTC (rev 4584)
+++ trunk/src/clients/reconcilio/command_line.hh 2008-04-20 01:21:35 UTC (rev 4585)
@@ -53,7 +53,6 @@
paludis::args::AliasArg a_no_color;
paludis::args::StringArg a_environment;
paludis::args::SwitchArg a_exact;
- paludis::args::SwitchArg a_verbose;
paludis::args::StringArg a_resume_command_template;
paludis::args::SwitchArg a_compact;
Modified: trunk/src/clients/reconcilio/fix_linkage.cc
===================================================================
--- trunk/src/clients/reconcilio/fix_linkage.cc 2008-04-20 00:25:22 UTC (rev 4584)
+++ trunk/src/clients/reconcilio/fix_linkage.cc 2008-04-20 01:21:35 UTC (rev 4585)
@@ -47,7 +47,6 @@
{
Context ctx("When performing the Fix Linkage action:");
- bool verbose(CommandLine::get_instance()->a_verbose.specified());
std::string library(CommandLine::get_instance()->a_library.argument());
if (library.empty())
@@ -70,16 +69,13 @@
std::cout << std::endl << colour(cl_heading, "Broken packages:") << std::endl;
else
std::cout << std::endl << colour(cl_heading, "Packages that depend on " + library + ":") << std::endl;
- if (! verbose)
- std::cout << std::endl;
}
tr1::shared_ptr<Sequence<std::string> > targets(new Sequence<std::string>);
for (BrokenLinkageFinder::BrokenPackageConstIterator pkg_it(finder.begin_broken_packages()),
pkg_it_end(finder.end_broken_packages()); pkg_it_end != pkg_it; ++pkg_it)
{
- if (verbose)
- std::cout << std::endl;
+ std::cout << std::endl;
std::string pkgname(stringify((*pkg_it)->name()));
std::string fullname((*pkg_it)->canonical_form(idcf_full));
@@ -88,18 +84,17 @@
fullname.replace(pos, pkgname.length(), colour(cl_package_name, pkgname));
std::cout << "* " << fullname << std::endl;
- if (verbose)
- for (BrokenLinkageFinder::BrokenFileConstIterator file_it(finder.begin_broken_files(*pkg_it)),
- file_it_end(finder.end_broken_files(*pkg_it)); file_it_end != file_it; ++file_it)
- {
- std::cout << " " << *file_it;
- if (library.empty())
- std::cout << " (requires "
- << join(finder.begin_missing_requirements(*pkg_it, *file_it),
- finder.end_missing_requirements(*pkg_it, *file_it),
- " ") << ")";
- std::cout << std::endl;
- }
+ for (BrokenLinkageFinder::BrokenFileConstIterator file_it(finder.begin_broken_files(*pkg_it)),
+ file_it_end(finder.end_broken_files(*pkg_it)); file_it_end != file_it; ++file_it)
+ {
+ std::cout << " " << *file_it;
+ if (library.empty())
+ std::cout << " (requires "
+ << join(finder.begin_missing_requirements(*pkg_it, *file_it),
+ finder.end_missing_requirements(*pkg_it, *file_it),
+ " ") << ")";
+ std::cout << std::endl;
+ }
if (CommandLine::get_instance()->a_exact.specified())
targets->push_back(stringify(make_package_dep_spec()
@@ -113,7 +108,7 @@
}
tr1::shared_ptr<const PackageID> orphans;
- if (verbose && finder.begin_broken_files(orphans) != finder.end_broken_files(orphans))
+ if (finder.begin_broken_files(orphans) != finder.end_broken_files(orphans))
{
if (library.empty())
std::cout << std::endl << "The following broken files are not owned by any installed package:" << std::endl;
More information about the paludis-commits
mailing list