summaryrefslogtreecommitdiff
path: root/ftparchive/writer.h
diff options
context:
space:
mode:
authorDavid Kalnischkies <david@kalnischkies.de>2015-10-28 14:38:49 +0100
committerDavid Kalnischkies <david@kalnischkies.de>2015-11-04 18:42:27 +0100
commit1dd20368486820efb6ef4476ad739e967174bec4 (patch)
treee47f748b70975ab3fadd5c10734156346b172e5e /ftparchive/writer.h
parente2ea6b63d333e6999d855a026a005726067c38ac (diff)
support arch:all data e.g. in separate Packages file
Based on a discussion with Niels Thykier who asked for Contents-all this implements apt trying for all architecture dependent files to get a file for the architecture all, which is treated internally now as an official architecture which is always around (like native). This way arch:all data can be shared instead of duplicated for each architecture requiring the user to download the same information again and again. There is one problem however: In Debian there is already a binary-all/ Packages file, but the binary-any files still include arch:all packages, so that downloading this file now would be a waste of time, bandwidth and diskspace. We therefore need a way to decide if it makes sense to download the all file for Packages in Debian or not. The obvious answer would be a special flag in the Release file indicating this, which would need to default to 'no' and every reasonable repository would override it to 'yes' in a few years time, but the flag would be there "forever". Looking closer at a Release file we see the field "Architectures", which doesn't include 'all' at the moment. With the idea outlined above that 'all' is a "proper" architecture now, we interpret this field as being authoritative in declaring which architectures are supported by this repository. If it says 'all', apt will try to get all, if not it will be skipped. This gives us another interesting feature: If I configure a source to download armel and mips, but it declares it supports only armel apt will now print a notice saying as much. Previously this was a very cryptic failure. If on the other hand the repository supports mips, too, but for some reason doesn't ship mips packages at the moment, this 'missing' file is silently ignored (= that is the same as the repository including an empty file). The Architectures field isn't mandatory through, so if it isn't there, we assume that every architecture is supported by this repository, which skips the arch:all if not listed in the release file.
Diffstat (limited to 'ftparchive/writer.h')
-rw-r--r--ftparchive/writer.h9
1 files changed, 6 insertions, 3 deletions
diff --git a/ftparchive/writer.h b/ftparchive/writer.h
index 98012beee..ea4c66da4 100644
--- a/ftparchive/writer.h
+++ b/ftparchive/writer.h
@@ -40,6 +40,7 @@ class FTWScanner
protected:
vector<string> Patterns;
string Arch;
+ bool IncludeArchAll;
const char *OriginalPath;
bool ErrorPrinted;
@@ -79,7 +80,7 @@ class FTWScanner
void AddPatterns(std::vector<std::string> const &patterns) { Patterns.insert(Patterns.end(), patterns.begin(), patterns.end()); };
bool SetExts(string const &Vals);
- FTWScanner(FileFd * const Output, string const &Arch = string());
+ FTWScanner(FileFd * const Output, string const &Arch = string(), bool const IncludeArchAll = true);
virtual ~FTWScanner();
};
@@ -125,7 +126,8 @@ class PackagesWriter : public FTWScanner
PackagesWriter(FileFd * const Output, TranslationWriter * const TransWriter, string const &DB,
string const &Overrides,
string const &ExtOverrides = "",
- string const &Arch = "");
+ string const &Arch = "",
+ bool const IncludeArchAll = true);
virtual ~PackagesWriter();
};
@@ -149,7 +151,8 @@ class ContentsWriter : public FTWScanner
void Finish() {Gen.Print(*Output);};
inline bool ReadyDB(string const &DB) {return Db.ReadyDB(DB);};
- ContentsWriter(FileFd * const Output, string const &DB, string const &Arch = string());
+ ContentsWriter(FileFd * const Output, string const &DB, string const &Arch = string(),
+ bool const IncludeArchAll = true);
virtual ~ContentsWriter() {};
};