aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAvatar Ciaran McCreesh <ciaran.mccreesh@googlemail.com> 2006-02-15 03:01:00 +0000
committerAvatar Ciaran McCreesh <ciaran.mccreesh@googlemail.com> 2006-02-15 03:01:00 +0000
commit5480757f20817cf5cbebed4c456645a7f8aa48d6 (patch)
tree99e047c903996cbedf542e9569cd9d72218aadc8
parent83c3a76db4f584e01f1290c5782677f5e3d7e5ea (diff)
downloadpaludis-5480757f20817cf5cbebed4c456645a7f8aa48d6.tar.gz
paludis-5480757f20817cf5cbebed4c456645a7f8aa48d6.tar.xz
Some crude checks for use? flags in *DEPEND. Currently gives various false matches.
-rw-r--r--paludis/qa/dep_flags_check.cc136
-rw-r--r--paludis/qa/dep_flags_check.hh51
-rw-r--r--paludis/qa/files.m41
3 files changed, 188 insertions, 0 deletions
diff --git a/paludis/qa/dep_flags_check.cc b/paludis/qa/dep_flags_check.cc
new file mode 100644
index 0000000..f6c4f2d
--- /dev/null
+++ b/paludis/qa/dep_flags_check.cc
@@ -0,0 +1,136 @@
+/* vim: set sw=4 sts=4 et foldmethod=syntax : */
+
+/*
+ * Copyright (c) 2006 Ciaran McCreesh <ciaranm@gentoo.org>
+ *
+ * This file is part of the Paludis package manager. Paludis is free software;
+ * you can redistribute it and/or modify it under the terms of the GNU General
+ * Public License version 2, as published by the Free Software Foundation.
+ *
+ * Paludis is distributed in the hope that it will be useful, but WITHOUT ANY
+ * WARRANTY; without even the implied warranty of MERCHANTABILITY or FITNESS
+ * FOR A PARTICULAR PURPOSE. See the GNU General Public License for more
+ * details.
+ *
+ * You should have received a copy of the GNU General Public License along with
+ * this program; if not, write to the Free Software Foundation, Inc., 59 Temple
+ * Place, Suite 330, Boston, MA 02111-1307 USA
+ */
+
+#include "dep_flags_check.hh"
+#include <paludis/dep_parser.hh>
+#include <paludis/dep_atom_visitor.hh>
+#include <paludis/any_dep_atom.hh>
+#include <paludis/all_dep_atom.hh>
+#include <paludis/package_dep_atom.hh>
+#include <paludis/block_dep_atom.hh>
+#include <paludis/use_dep_atom.hh>
+
+using namespace paludis;
+using namespace paludis::qa;
+
+namespace
+{
+ struct Checker :
+ DepAtomVisitorTypes::ConstVisitor
+ {
+ CheckResult & result;
+ const std::string role;
+ const std::set<UseFlagName> & iuse;
+
+ Checker(CheckResult & rr, const std::string & r, const std::set<UseFlagName> & i) :
+ result(rr),
+ role(r),
+ iuse(i)
+ {
+ }
+
+ void visit(const PackageDepAtom * const)
+ {
+ }
+
+ void visit(const AllDepAtom * const a)
+ {
+ std::for_each(a->begin(), a->end(), accept_visitor(this));
+ }
+
+ void visit(const AnyDepAtom * const a)
+ {
+ std::for_each(a->begin(), a->end(), accept_visitor(this));
+ }
+
+ void visit(const UseDepAtom * const u)
+ {
+ /// \bug VV arch flags should be qal_maybe and a different message
+ if (iuse.end() == iuse.find(u->flag()))
+ result << Message(qal_major, "Conditional flag '" + stringify(u->flag()) +
+ "' in " + role + " not in IUSE");
+
+ std::for_each(u->begin(), u->end(), accept_visitor(this));
+ }
+
+ void visit(const BlockDepAtom * const)
+ {
+ }
+ };
+}
+
+DepFlagsCheck::DepFlagsCheck()
+{
+}
+
+CheckResult
+DepFlagsCheck::operator() (const EbuildCheckData & e) const
+{
+ CheckResult result(stringify(e.get<ecd_name>()) + "-" + stringify(e.get<ecd_version>()),
+ identifier());
+
+ try
+ {
+ PackageDatabaseEntry ee(e.get<ecd_name>(), e.get<ecd_version>(),
+ e.get<ecd_environment>()->package_database()->favourite_repository());
+ VersionMetadata::ConstPointer metadata(
+ e.get<ecd_environment>()->package_database()->fetch_metadata(ee));
+
+ std::set<UseFlagName> iuse(metadata->begin_iuse(), metadata->end_iuse());
+ /// \todo VV need a more elegant way of doing this
+ iuse.insert(UseFlagName("bootstrap"));
+ iuse.insert(UseFlagName("build"));
+ iuse.insert(UseFlagName("elibc_glibc"));
+ iuse.insert(UseFlagName("userland_Darwin"));
+ iuse.insert(UseFlagName("userland_bsd"));
+ iuse.insert(UseFlagName("userland_GNU"));
+
+ Checker depend_checker(result, "DEPEND", iuse);
+ std::string depend(metadata->get(vmk_depend));
+ DepParser::parse(depend)->accept(&depend_checker);
+
+ Checker rdepend_checker(result, "RDEPEND", iuse);
+ std::string rdepend(metadata->get(vmk_rdepend));
+ DepParser::parse(rdepend)->accept(&rdepend_checker);
+
+ Checker pdepend_checker(result, "PDEPEND", iuse);
+ std::string pdepend(metadata->get(vmk_pdepend));
+ DepParser::parse(pdepend)->accept(&pdepend_checker);
+ }
+ catch (const InternalError &)
+ {
+ throw;
+ }
+ catch (const Exception & e)
+ {
+ result << Message(qal_fatal, "Caught Exception '" + e.message() + "' ("
+ + e.what() + ")");
+ }
+
+ return result;
+}
+
+const std::string &
+DepFlagsCheck::identifier()
+{
+ static const std::string id("dep flags");
+ return id;
+}
+
+
diff --git a/paludis/qa/dep_flags_check.hh b/paludis/qa/dep_flags_check.hh
new file mode 100644
index 0000000..c380302
--- /dev/null
+++ b/paludis/qa/dep_flags_check.hh
@@ -0,0 +1,51 @@
+/* vim: set sw=4 sts=4 et foldmethod=syntax : */
+
+/*
+ * Copyright (c) 2006 Ciaran McCreesh <ciaranm@gentoo.org>
+ *
+ * This file is part of the Paludis package manager. Paludis is free software;
+ * you can redistribute it and/or modify it under the terms of the GNU General
+ * Public License version 2, as published by the Free Software Foundation.
+ *
+ * Paludis is distributed in the hope that it will be useful, but WITHOUT ANY
+ * WARRANTY; without even the implied warranty of MERCHANTABILITY or FITNESS
+ * FOR A PARTICULAR PURPOSE. See the GNU General Public License for more
+ * details.
+ *
+ * You should have received a copy of the GNU General Public License along with
+ * this program; if not, write to the Free Software Foundation, Inc., 59 Temple
+ * Place, Suite 330, Boston, MA 02111-1307 USA
+ */
+
+#ifndef PALUDIS_GUARD_PALUDIS_QA_DEP_FLAGS_CHECK_HH
+#define PALUDIS_GUARD_PALUDIS_QA_DEP_FLAGS_CHECK_HH 1
+
+#include <paludis/qa/ebuild_check.hh>
+#include <string>
+
+namespace paludis
+{
+ namespace qa
+ {
+ class DepFlagsCheck :
+ public EbuildCheck
+ {
+ public:
+ DepFlagsCheck();
+
+ CheckResult operator() (const EbuildCheckData &) const;
+
+ static const std::string & identifier();
+
+ virtual std::string describe() const
+ {
+ return "Checks *DEPEND and PROVIDE use? flags";
+ }
+ };
+
+ static const EbuildCheckMaker::RegisterMaker register_dep_flags_check(
+ DepFlagsCheck::identifier(), &make_ebuild_check<DepFlagsCheck>);
+ }
+}
+
+#endif
diff --git a/paludis/qa/files.m4 b/paludis/qa/files.m4
index f288967..1e9ff43 100644
--- a/paludis/qa/files.m4
+++ b/paludis/qa/files.m4
@@ -12,6 +12,7 @@ add(`changelog_check', `hh', `cc')
add(`check', `hh', `cc')
add(`check_result', `hh', `cc', `test')
add(`create_metadata_check', `hh', `cc')
+add(`dep_flags_check', `hh', `cc')
add(`deps_exist_check', `hh', `cc')
add(`description_check', `hh', `cc')
add(`ebuild_check', `hh', `cc')