From f9e186ab704401416832d723c2e70a0f7a7435b3 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Petr=20Mr=C3=A1zek?= Date: Wed, 27 May 2015 01:30:18 +0200 Subject: GH-967 make libraries handle their own path prefix Makes it possible to mix libraries managed by FTB and MultiMC Backport from unstable --- logic/forge/ForgeInstaller.cpp | 4 +-- logic/ftb/FTBProfileStrategy.cpp | 2 ++ logic/ftb/OneSixFTBInstance.cpp | 4 +-- logic/minecraft/OneSixInstance.cpp | 4 +-- logic/minecraft/OneSixLibrary.cpp | 1 + logic/minecraft/OneSixUpdate.cpp | 4 +-- logic/minecraft/RawLibrary.cpp | 54 +++++++++++++++++++++++++++++++------- logic/minecraft/RawLibrary.h | 36 ++++++++++++++++++------- logic/minecraft/VersionFile.cpp | 2 +- 9 files changed, 82 insertions(+), 29 deletions(-) (limited to 'logic') diff --git a/logic/forge/ForgeInstaller.cpp b/logic/forge/ForgeInstaller.cpp index a2a9c863..18527c49 100644 --- a/logic/forge/ForgeInstaller.cpp +++ b/logic/forge/ForgeInstaller.cpp @@ -87,8 +87,8 @@ void ForgeInstaller::prepare(const QString &filename, const QString &universalUr // where do we put the library? decode the mojang path OneSixLibrary lib(libraryName); - auto cacheentry = ENV.metacache()->resolveEntry("libraries", lib.storagePath()); - finalPath = "libraries/" + lib.storagePath(); + auto cacheentry = ENV.metacache()->resolveEntry("libraries", lib.storageSuffix()); + finalPath = "libraries/" + lib.storageSuffix(); if (!ensureFilePathExists(finalPath)) return; diff --git a/logic/ftb/FTBProfileStrategy.cpp b/logic/ftb/FTBProfileStrategy.cpp index ff40b642..293d0fdc 100644 --- a/logic/ftb/FTBProfileStrategy.cpp +++ b/logic/ftb/FTBProfileStrategy.cpp @@ -41,6 +41,7 @@ void FTBProfileStrategy::loadDefaultBuiltinPatches() } profile->appendPatch(minecraftPatch); + auto nativeInstance = dynamic_cast(m_instance); ProfilePatchPtr packPatch; { auto mcJson = m_instance->minecraftRoot() + "/pack.json"; @@ -58,6 +59,7 @@ void FTBProfileStrategy::loadDefaultBuiltinPatches() { addLib->m_hint = "local"; addLib->insertType = RawLibrary::Prepend; + addLib->setStoragePrefix(nativeInstance->librariesPath().absolutePath()); } file->fileId = "org.multimc.ftb.pack"; file->setVanilla(true); diff --git a/logic/ftb/OneSixFTBInstance.cpp b/logic/ftb/OneSixFTBInstance.cpp index a8ea3031..88c885b9 100644 --- a/logic/ftb/OneSixFTBInstance.cpp +++ b/logic/ftb/OneSixFTBInstance.cpp @@ -78,7 +78,7 @@ void OneSixFTBInstance::copy(const QDir &newDir) for (auto library : libraryNames) { OneSixLibrary *lib = new OneSixLibrary(library); - const QString out = QDir::current().absoluteFilePath("libraries/" + lib->storagePath()); + const QString out = QDir::current().absoluteFilePath("libraries/" + lib->storageSuffix()); if (QFile::exists(out)) { continue; @@ -87,7 +87,7 @@ void OneSixFTBInstance::copy(const QDir &newDir) { qCritical() << "Couldn't create folder structure for" << out; } - if (!QFile::copy(librariesPath().absoluteFilePath(lib->storagePath()), out)) + if (!QFile::copy(librariesPath().absoluteFilePath(lib->storageSuffix()), out)) { qCritical() << "Couldn't copy" << lib->rawName(); } diff --git a/logic/minecraft/OneSixInstance.cpp b/logic/minecraft/OneSixInstance.cpp index c551dc9b..b2b8698f 100644 --- a/logic/minecraft/OneSixInstance.cpp +++ b/logic/minecraft/OneSixInstance.cpp @@ -139,7 +139,7 @@ BaseProcess *OneSixInstance::prepareForLaunch(AuthSessionPtr session) auto libs = m_version->getActiveNormalLibs(); for (auto lib : libs) { - launchScript += "cp " + librariesPath().absoluteFilePath(lib->storagePath()) + "\n"; + launchScript += "cp " + QFileInfo(lib->storagePath()).absoluteFilePath() + "\n"; } auto jarMods = getJarMods(); if (!jarMods.isEmpty()) @@ -191,7 +191,7 @@ BaseProcess *OneSixInstance::prepareForLaunch(AuthSessionPtr session) QDir natives_dir(PathCombine(instanceRoot(), "natives/")); for (auto native : m_version->getActiveNativeLibs()) { - QFileInfo finfo(PathCombine("libraries", native->storagePath())); + QFileInfo finfo(native->storagePath()); launchScript += "ext " + finfo.absoluteFilePath() + "\n"; } launchScript += "natives " + natives_dir.absolutePath() + "\n"; diff --git a/logic/minecraft/OneSixLibrary.cpp b/logic/minecraft/OneSixLibrary.cpp index 43d381fb..1ebba507 100644 --- a/logic/minecraft/OneSixLibrary.cpp +++ b/logic/minecraft/OneSixLibrary.cpp @@ -29,6 +29,7 @@ OneSixLibrary::OneSixLibrary(RawLibraryPtr base) m_native_classifiers = base->m_native_classifiers; m_rules = base->m_rules; dependType = base->dependType; + m_storagePrefix = base->m_storagePrefix; // these only make sense for raw libraries. OneSix /* insertType = base->insertType; diff --git a/logic/minecraft/OneSixUpdate.cpp b/logic/minecraft/OneSixUpdate.cpp index 24ee5804..9dcb977b 100644 --- a/logic/minecraft/OneSixUpdate.cpp +++ b/logic/minecraft/OneSixUpdate.cpp @@ -227,8 +227,8 @@ void OneSixUpdate::jarlibStart() continue; } - QString raw_storage = lib->storagePath(); - QString raw_dl = lib->downloadUrl(); + QString raw_storage = lib->storageSuffix(); + QString raw_dl = lib->url().toString(); auto f = [&](QString storage, QString dl) { diff --git a/logic/minecraft/RawLibrary.cpp b/logic/minecraft/RawLibrary.cpp index 3c771895..fa8270cd 100644 --- a/logic/minecraft/RawLibrary.cpp +++ b/logic/minecraft/RawLibrary.cpp @@ -2,6 +2,7 @@ using namespace MMCJson; #include "RawLibrary.h" +#include RawLibraryPtr RawLibrary::fromJson(const QJsonObject &libObj, const QString &filename) { @@ -29,7 +30,9 @@ RawLibraryPtr RawLibrary::fromJson(const QJsonObject &libObj, const QString &fil return true; }; - readString("url", out->m_base_url); + QString urlStr; + readString("url", urlStr); + out->m_base_url = urlStr; readString("MMC-hint", out->m_hint); readString("MMC-absulute_url", out->m_absolute_url); readString("MMC-absoluteUrl", out->m_absolute_url); @@ -109,7 +112,7 @@ RawLibraryPtr RawLibrary::fromJsonPlus(const QJsonObject &libObj, const QString throw JSONValidationError("Empty compound insert rule in " + filename); } QString insertString = insertObj.keys().first(); - // really, only replace makes sense in combination with + // really, only replace makes sense in combination with if(insertString != "replace") { throw JSONValidationError("Compound insert rule is not 'replace' in " + filename); @@ -154,7 +157,7 @@ QJsonObject RawLibrary::toJson() const m_base_url != "https://" + URLConstants::AWS_DOWNLOAD_LIBRARIES && m_base_url != "https://" + URLConstants::LIBRARY_BASE && !m_base_url.isEmpty()) { - libRoot.insert("url", m_base_url); + libRoot.insert("url", m_base_url.toString()); } if (isNative()) { @@ -194,7 +197,7 @@ QJsonObject RawLibrary::toJson() const QStringList RawLibrary::files() const { QStringList retval; - QString storage = storagePath(); + QString storage = storageSuffix(); if (storage.contains("${arch}")) { QString cooked_storage = storage; @@ -221,17 +224,19 @@ bool RawLibrary::filesExist(const QDir &base) const } return true; } -QString RawLibrary::downloadUrl() const +QUrl RawLibrary::url() const { - if (m_absolute_url.size()) + if (!m_absolute_url.isEmpty()) + { return m_absolute_url; + } if (m_base_url.isEmpty()) { - return QString("https://" + URLConstants::LIBRARY_BASE) + storagePath(); + return QString("https://" + URLConstants::LIBRARY_BASE) + storageSuffix(); } - return m_base_url + storagePath(); + return m_base_url.resolved(storageSuffix()); } bool RawLibrary::isActive() const @@ -259,7 +264,26 @@ bool RawLibrary::isActive() const return result; } -QString RawLibrary::storagePath() const +void RawLibrary::setStoragePrefix(QString prefix) +{ + m_storagePrefix = prefix; +} + +QString RawLibrary::defaultStoragePrefix() +{ + return "libraries/"; +} + +QString RawLibrary::storagePrefix() const +{ + if(m_storagePrefix.isEmpty()) + { + return defaultStoragePrefix(); + } + return m_storagePrefix; +} + +QString RawLibrary::storageSuffix() const { // non-native? use only the gradle specifier if (!isNative()) @@ -269,7 +293,7 @@ QString RawLibrary::storagePath() const // otherwise native, override classifiers. Mojang HACK! GradleSpecifier nativeSpec = m_name; - if(m_native_classifiers.contains(currentSystem)) + if (m_native_classifiers.contains(currentSystem)) { nativeSpec.setClassifier(m_native_classifiers[currentSystem]); } @@ -279,3 +303,13 @@ QString RawLibrary::storagePath() const } return nativeSpec.toPath(); } + +QString RawLibrary::storagePath() const +{ + return PathCombine(storagePrefix(), storageSuffix()); +} + +bool RawLibrary::storagePathIsDefault() const +{ + return m_storagePrefix.isEmpty(); +} diff --git a/logic/minecraft/RawLibrary.h b/logic/minecraft/RawLibrary.h index a3990740..3f3cb925 100644 --- a/logic/minecraft/RawLibrary.h +++ b/logic/minecraft/RawLibrary.h @@ -5,6 +5,7 @@ #include #include #include +#include #include #include "minecraft/OneSixRule.h" @@ -33,7 +34,7 @@ public: /* methods */ { return m_name; } - + void setRawName(const GradleSpecifier & spec) { m_name = spec; @@ -43,7 +44,7 @@ public: /* methods */ { m_name.setClassifier(spec); } - + /// returns the full group and artifact prefix QString artifactPrefix() const { @@ -68,8 +69,24 @@ public: /* methods */ return m_native_classifiers.size() != 0; } + void setStoragePrefix(QString prefix = QString()); + + /// the default storage prefix used by MultiMC + static QString defaultStoragePrefix(); + + bool storagePathIsDefault() const; + + /// Get the prefix - root of the storage to be used + QString storagePrefix() const; + + /// Get the relative path where the library should be saved + QString storageSuffix() const; + + /// Get the absolute path where the library should be saved + QString storagePath() const; + /// Set the url base for downloads - void setBaseUrl(const QString &base_url) + void setBaseUrl(const QUrl &base_url) { m_base_url = base_url; } @@ -110,11 +127,7 @@ public: /* methods */ bool isActive() const; /// Get the URL to download the library from - QString downloadUrl() const; - - /// Get the relative path where the library should be saved - QString storagePath() const; - + QUrl url() const; protected: /* data */ /// the basic gradle dependency specifier. @@ -128,7 +141,7 @@ protected: /* data */ public: /* data */ // TODO: make all of these protected, clean up semantics of implicit vs. explicit values. /// URL where the file can be downloaded - QString m_base_url; + QUrl m_base_url; /// DEPRECATED: absolute URL. takes precedence the normal download URL, if defined QString m_absolute_url; @@ -136,6 +149,9 @@ public: /* data */ /// type hint - modifies how the library is treated QString m_hint; + /// storage - by default the local libraries folder in multimc, but could be elsewhere + QString m_storagePrefix; + /// true if the library had an extract/excludes section (even empty) bool applyExcludes = false; @@ -160,7 +176,7 @@ public: /* data */ Replace } insertType = Append; QString insertData; - + /// determines how can libraries be applied. conflicting dependencies cause errors. enum DependType { diff --git a/logic/minecraft/VersionFile.cpp b/logic/minecraft/VersionFile.cpp index 0a20efcb..6ef5a8bc 100644 --- a/logic/minecraft/VersionFile.cpp +++ b/logic/minecraft/VersionFile.cpp @@ -369,7 +369,7 @@ void VersionFile::applyTo(MinecraftProfile *version) if (index >= 0) { auto existingLibrary = version->libraries[index]; - if (!addedLibrary->m_base_url.isNull()) + if (!addedLibrary->m_base_url.isEmpty()) { existingLibrary->setBaseUrl(addedLibrary->m_base_url); } -- cgit v1.2.3