| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
|
|
|
| |
If dpkg-gencontrol was involved in the creation of a package we will not
usually encounter empty or otherwise useless fields, but apparently not
everyone is using it.
It isn't recommended to have these empty lines, but it isn't too hard to
ignore for Provides as we did for dependencies already and apt-ftparchive
can be convinced to produce empty files (if you feed it such a package)
as well, so lets be nice and provide users with a more accepting parser.
Closes: #1069874
|
|\
| |
| |
| |
| | |
Split out of mostly independent fixes: cmake execute errors, removed rev-deps, protected garbage & co
See merge request apt-team/apt!345
|
| |
| |
| |
| |
| | |
Our code does the right thing currently, so lets add a test to ensure
this keeps being the case in the future.
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Our autoremoval-advertisment is modified by SUDO_USER as if the current
apt call was made with sudo it seems a good idea to show the ad with
sudo as well. That is annoying for our tests through as normally the
tests are run locally or by autopkgtest without sudo, but in Gitlab CI
we use it (to run our tests as user… as we are already root) and so
individual tests had to deal with this.
That is annoying and really not needed as we can have our autoremove
test check that this ad gets displayed the right way and ignore it the
rest of the time.
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
We schedule reverse dependencies for an upgrade, but we shouldn't do it
if we have ear-marked this package for removal later on. Usually the
solver will end up doing the right thing like it already did in the
included testcase in the end, but given that before it reaches the right
end it explored a bad path which can lead to more installs and removals
influencing later decisions or are just too hard for the resolver to
undo later on, we can just not explore this path to begin with.
References: e077370ffcb3669a50a600e80356c2002e6b176d
|
| |
| |
| |
| |
| |
| |
| |
| |
| | |
We remove new garbage packages from the solution if we can as installing
a new package which is at the same time considered garbage looks silly,
but it could also be a new dependency of another garbage package, so we
have a second round trying to save such packages. In this round we
weren't considering versioned constraints on dependency relations through
so even an unsatisfied old recommends could save which it shouldn't.
|
| |
| |
| |
| |
| |
| |
| | |
Not all commands show kept back packages, so even if it hardly matters,
lets not crunch numbers needlessly if we don't need the info which also
reduces the lifetime of the involved variables hopefully also reducing
the mental requirements for a reader.
|
|/
|
|
|
|
| |
Ignoring errors might lead to failures later on anyhow, but especially
with triehash it could also lead to broken builds or other crazy stuff,
so lets better be save than sorry.
|
|
|
|
|
| |
This turned out to be a bit too bold for most of them, given their
informational nature.
|
| |
|
| |
|
| |
|
|\
| |
| |
| |
| | |
UI changes for 2.9.2
See merge request apt-team/apt!343
|
| |
| |
| |
| |
| |
| | |
This avoids awkward interactions with Installing dependencies and
Suggested packages appearing before it; despite being triggered by
it.
|
| | |
|
| |
| |
| |
| |
| |
| | |
Add a new `note` field to `ShowList` which contains a line to
be displayed after the list itself. This allows `ShowList` to
render a self-contained list block.
|
| |
| |
| |
| | |
We should pass this properly to the TagSection.write()
|
| | |
|
| |
| |
| |
| |
| | |
This allows these bits to be localized, improving user experience
for users in different languages.
|
| |
| |
| |
| |
| | |
This draws a bit more attention, and improves readability vs
keeping the color for the entire message.
|
| |
| |
| |
| | |
Yellow is a bit odd for notices.
|
| |
| |
| |
| |
| | |
GTest has a lot of broken things with signed vs unsigned,
and double integer promotions, let's silence them.
|
| |
| |
| |
| |
| |
| | |
This avoids us polluting the configuration for later tests, since
the test order apparently is not deterministic. We probably should
fix some common test case thingy instead.
|
|\ \
| |/
|/|
| |
| | |
Add an "audit" error level
See merge request apt-team/apt!323
|
| | |
|
| |
| |
| |
| |
| | |
This option is recommended to be used by repository operators
for testing.
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
We never used the debug level before, so we can do that. This
allows us to have the new audit level.
We did call DumpErrors() with DEBUG in two debug code paths,
so don't touch those.
debug
|
|/
|
|
|
| |
This will aggressively highlight out-of-compliance vs the best
practices.
|
|\
| |
| |
| |
| | |
Introduce APT::Configuration::Color, --color option and documentation for color.
See merge request apt-team/apt!342
|
| | |
|
| |
| |
| |
| | |
This is somewhat pointless.
|
| |
| |
| |
| | |
Wrap empty else in {} to avoid readability issues
|
| | |
|
| | |
|
|/ |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
All other entries in a dependency line get substantial leeway about the
amount of spaces surrounding the entry itself and its individual parts,
but the very last entry was required to have a version constraint be
at least 4 chars long (excluding opening bracket and spaces following
it), so if the version is short and a single-char relation used a space
had to make up for it. This is a bit unfair in comparison to the other
entries who do not have such unreasonable demands, so we reduce our
demand to 3 chars or longer, which is satisfied by "=1)".
If it is a good idea to hate spaces that much remains unanswered by this
commit, but in practice most tools (re)writing the files we parse will
include spaces, so its only in files (or on the satisfy command line)
directly edited by users that we can encounter such a situation, which
is a relatively new development given this line came unchanged from
the introduction of this method in 1998.
LP: #2061834
|
|
|
|
| |
Closes: #1069118
|
|
|
|
|
|
|
| |
Note by committer: Merged with latest pot and resolving 3 white space
fuzzies introduced by it
Closes: #1068943
|
|
|
|
|
| |
While this is an interactive only change that doesn't break
parsing someone got unhappy.
|
|
|
|
|
| |
This is easier to discover than APT::Color options and matches
best practices.
|
| |
|
|\
| |
| |
| |
| | |
UI improvements for 2.9.1
See merge request apt-team/apt!338
|
| |
| |
| |
| |
| | |
This produces a much more appealing progress bar and it can even
show parts of the progress being done.
|
| |
| |
| |
| |
| | |
This especially helps with the running out of space warnings to
be safe.
|
| | |
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
dnf shows downgrades as green because the package comes from a remote
repository, but I think yellow is more indicative that they are
unsupported.
Ganneff asked for yellow for kept packages too, however I'm not
adding yellow there, as kept packages are more indicative of an
archive error, likely temporary, rather than an action item for
the user.
|
| | |
|
| |
| |
| |
| |
| |
| | |
This aligns the /boot message with the parent one, and also align
the formatting of the partially downloaded size message to add spaces
around /
|
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Calculate an estimate of 110% of the biggest initrd + system.map
as the additional space a kernel needs in /boot.
If /boot is a different file system than /usr, print the size of
the kernels + the additional space they will need separately;
otherwise include it in our /usr figure.
|
| |
| |
| |
| |
| |
| |
| |
| | |
Also allow us to have utf-8 characters in xgettext, but msgcomm still
fails so ugh.
This avoids translations messing up the formatting a bit more, at
least, and makes apt build again
|