apt/apt-private, branch 2.9.20Debians commandline package managerhttps://git.kalnischkies.de/apt/atom?h=2.9.202024-12-19T17:24:37Zapt show: Add pinning information for --full2024-12-19T17:24:37ZJulian Andres Klodejulian.klode@canonical.com2024-10-29T13:06:51Zurn:sha1:71727c2a761889855b3c1aaf4243649e98032f67
Add new fields `APT-Pin`, `APT-Candidate`, and `APT-Release` to the
output, same as in EDSP (limited to one release for now).
Be more strict about which PAGER to execvp() directly2024-12-19T16:56:49ZJulian Andres Klodejulian.klode@canonical.com2024-12-19T16:49:59Zurn:sha1:f6198e45b4cd924f390f977077d96b4545da0c91
There could be | or ; or other stuff in PAGER that needs
special handling, let's encode a list of known-good characters
instead. This covers all known pagers.
You could also try to execvp() PAGER in its entirety and if
it fails with ENOENT fall back to the shell but then you
don't get correct error handling if the pager really doesn't
exist (in which case we should abort trying to page).
Introduce automatic pager for read commands2024-12-18T09:25:24ZJulian Andres Klodejak@debian.org2024-12-16T16:05:40Zurn:sha1:6e260e26e4cd671f781151e24d89a17b29a55530
Automatically show the output of `show`, `policy`, `list`,
`search`, `showsrc` in a pager.
The pager setup is inspired by git's pager setup. Notably,
the pager is found using APT_PAGER and PAGER variables.
We wait for the pager to be setup somewhat correctly by
using a notify pipe to figure out whether execvp() was
succesful - then the pipe will read EOF as the other end
got closed by CLOEXEC during exec - or not, then the pipe
will contain an errno.
We set up the correct handlers for signals and exit to close
the fds and wait for the pager. Notably inside the signal
handler we cannot flush our streams, only close them, so
there is some duplication.
We call the InitOutputPager() function from inside the
various Do...() functions rather than setting it up
generally in InitOutput(). Doing so allows us to first
render the progress without a pager, and then setup
the pager for the content only which improves user
experience.
When we setup a pager we also take care to disable
standard input, as we should not be prompting users
while a pager is running (the pager will be reading
from the tty directly). We do this by dup2-ing() a
/dev/null over it; if we just close()d the fd, another
open() might reuse the fd number and problems could
occur.
Collect unprinted Ign errors for display in Err output2024-11-22T20:05:35ZDavid Kalnischkiesdavid@kalnischkies.de2024-11-22T17:59:48Zurn:sha1:a732f74400147a9993358b4778335715a0d5e20e
Instead of deciding if the first or the last error in a retry chain is
the best one to display if the entire chain fails in the end we teach
our progress reporting to collect the ignored errors (if it hasn't shown
them) and print them all together in the Err line.
We de-duplicate the collection which makes it harder to match the Ign
line with the error messages, but for these cases its probably better to
enable direct display of the text anyhow – and repeating the same message
multiple times is boring and probably not that helpful for a user
wanting to know why all the downloads failed even they asked apt to
retry 429 times…
References: 841243df877b39dfcb907a7fab9c4e8b4f65d715
Add a virtual destructor to private CacheSet Matcher2024-11-22T12:11:13ZDavid Kalnischkiesdavid@kalnischkies.de2024-09-11T19:52:26Zurn:sha1:0fb71e375363f2b45b033bb2b324e80214325c61
This silences the warning:
| warning: ‘class Matcher’ has virtual functions and accessible non-virtual destructor [-Wnon-virtual-dtor]
Acknowledge non-erase usage of remove_if for volatile sources2024-11-22T12:11:13ZDavid Kalnischkiesdavid@kalnischkies.de2024-09-11T16:57:04Zurn:sha1:fa19a9ddbdddeaed44480ce7dade11d526336435
gcc produces a length warning starting with:
| warning: ignoring return value of ‘_FIter std::remove_if(_FIter, _FIter, _Predicate) [with …]’, declared with attribute ‘nodiscard’ [-Wunused-result]
which is usually correct, but in the usage here we don't want to call an
erase as we operate on a c-style new[] array here which contains pointers
into argv and has an explicit sentinel (nullptr) – FileSize() is based
on this sentinel.
Casting to void silences the warning as is standard practice and
hopefully indicates better that this is intended than ignoring the
warning for the casual on-looker.
Add a --comment option to record Comment: in history2024-11-18T19:15:08ZJulian Andres Klodejulian.klode@canonical.com2024-11-18T19:07:02Zurn:sha1:1c4fa81cc398e21868ad237509f1ea8e772d76de
Suggested-by: mhoye on IRC
Merge branch 'svequiv' into 'main'2024-11-14T14:54:40ZJulian Andres Klodejak@debian.org2024-11-14T14:54:40Zurn:sha1:82752f6e0c65541244c2607e5dedf437701e07dc
Internally replace APT::StringView with std::string_view where it doesn't affect the ABI, with implementations that support both std::string_view and APT::StringView with new shims. ReportMirrorFailureToCentral: fix use-after-free
See merge request apt-team/apt!394Merge branch 'json-config' into 'main'2024-11-14T14:38:32ZJulian Andres Klodejak@debian.org2024-11-14T14:38:32Zurn:sha1:8f10ee850db3892bc979694864451be3a73ad1e8
JSON: Pass options to hooks in hello message
See merge request apt-team/apt!389apt-private/private-update.cc: APT::StringView -> std::string_view2024-11-12T19:36:17Zнабnabijaczleweli@nabijaczleweli.xyz2024-11-12T16:59:25Zurn:sha1:e33add3c2afa33a6f81b2eaed1fdcd29dc28790f