diff options
| author | Bjoern Michaelsen <bjoern.michaelsen@canonical.com> | 2014-04-04 18:47:28 +0200 |
|---|---|---|
| committer | Fridrich Strba <fridrich@documentfoundation.org> | 2014-04-30 14:12:54 +0000 |
| commit | fb36190cd34683270b0eae345401ebaf8cf30e24 (patch) | |
| tree | 8400963223e9389af7bc887a1df0a7d3f15115b7 | |
| parent | f812ad553ab1702424357495c36cee2f2619e2c1 (diff) | |
ignore legacy OpenOffice.org-*-version, when a LibreOffice-*-version is given
Change-Id: Ie5b6ffee07b6e5ebf922bad1b4a268ff636412c9
Reviewed-on: https://gerrit.libreoffice.org/8853
Reviewed-by: Björn Michaelsen <bjoern.michaelsen@canonical.com>
Tested-by: Björn Michaelsen <bjoern.michaelsen@canonical.com>
(cherry picked from commit 42905aecfcdee9eb2c184bceeec4f8688056cd73)
Reviewed-on: https://gerrit.libreoffice.org/9209
Reviewed-by: Christian Lohmaier <lohmaier+LibreOffice@googlemail.com>
Reviewed-by: Michael Meeks <michael.meeks@collabora.com>
Reviewed-by: Fridrich Strba <fridrich@documentfoundation.org>
Tested-by: Fridrich Strba <fridrich@documentfoundation.org>
| -rw-r--r-- | desktop/source/deployment/misc/dp_dependencies.cxx | 15 |
1 files changed, 13 insertions, 2 deletions
diff --git a/desktop/source/deployment/misc/dp_dependencies.cxx b/desktop/source/deployment/misc/dp_dependencies.cxx index 817c9b4684c1..696d3a7ee77e 100644 --- a/desktop/source/deployment/misc/dp_dependencies.cxx +++ b/desktop/source/deployment/misc/dp_dependencies.cxx @@ -103,18 +103,29 @@ check(dp_misc::DescriptionInfoset const & infoset) { css::uno::Sequence< css::uno::Reference< css::xml::dom::XElement > > unsatisfied(n); sal_Int32 unsat = 0; + // check first if minimalVersionLibreOffice is specified -- in that case ignore the legacy OOo dependencies + bool bIgnoreOoo = false; + for (sal_Int32 i = 0; i < n; ++i) { + css::uno::Reference< css::xml::dom::XElement > e( + deps->item(i), css::uno::UNO_QUERY_THROW); + if ( e->getNamespaceURI() == namespaceLibreOffice && e->getTagName() == minimalVersionLibreOffice) + { + bIgnoreOoo = true; + break; + } + } for (sal_Int32 i = 0; i < n; ++i) { css::uno::Reference< css::xml::dom::XElement > e( deps->item(i), css::uno::UNO_QUERY_THROW); bool sat = false; if ( e->getNamespaceURI() == namespaceOpenOfficeOrg && e->getTagName() == minimalVersionOpenOfficeOrg ) { - sat = satisfiesMinimalVersion( + sat = bIgnoreOoo || satisfiesMinimalVersion( getReferenceOpenOfficeOrgMajorMinor(), e->getAttribute("value")); } else if ( e->getNamespaceURI() == namespaceOpenOfficeOrg && e->getTagName() == maximalVersionOpenOfficeOrg ) { - sat = satisfiesMaximalVersion( + sat = bIgnoreOoo || satisfiesMaximalVersion( getReferenceOpenOfficeOrgMajorMinor(), e->getAttribute("value")); } else if (e->getNamespaceURI() == namespaceLibreOffice && e->getTagName() == minimalVersionLibreOffice ) |