all lists on lists.proxmox.com
 help / color / mirror / Atom feed
From: Fabian Ebner <f.ebner@proxmox.com>
To: pve-devel@lists.proxmox.com
Subject: [pve-devel] [PATCH-SERIES proxmox-apt/pve-rs] better detection of standard repositories
Date: Thu, 29 Jul 2021 14:25:47 +0200	[thread overview]
Message-ID: <20210729122554.148980-1-f.ebner@proxmox.com> (raw)

by also requiring the suite to match. Let the caller pass along the
current suite instead of auto-detecting, and have the backend only
handle static stuff (avoids the need for Result<>).

Also refactor check_suites in a similar fashion, so it's more uniform
and so that get_current_release_codename only needs to be called once,
by the library user.

The pve-rs patch #1 is required after proxmox-apt patch #1 is applied.
The pve-rs patch #2 is required after proxmox-apt patches #3 and #5
are applied.

I'll send the corresponding patches for pmg-rs and proxmox-backup
separately.


proxmox-apt:

Fabian Ebner (5):
  standard repos: add suite parameter for stricter detection
  repo: make suite_variant helper more general
  check repos: have caller specify the current suite
  repo: remove has_suite_variant helper
  add type DebianCodename

 src/repositories/file.rs       |  84 ++++++++++++++-------------
 src/repositories/mod.rs        |  31 +++++-----
 src/repositories/release.rs    | 103 ++++++++++++++++++++++++++-------
 src/repositories/repository.rs |  33 ++++-------
 tests/repositories.rs          |  27 ++++++---
 5 files changed, 171 insertions(+), 107 deletions(-)


pve-rs:

Fabian Ebner (2):
  apt: repos: adapt to back-end changes
  apt: repos: adapt to further back-end changes

 src/apt/repositories.rs | 11 +++++++----
 1 file changed, 7 insertions(+), 4 deletions(-)

-- 
2.30.2





             reply	other threads:[~2021-07-29 12:26 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-07-29 12:25 Fabian Ebner [this message]
2021-07-29 12:25 ` [pve-devel] [PATCH proxmox-apt 1/5] standard repos: add suite parameter for stricter detection Fabian Ebner
2021-07-29 12:25 ` [pve-devel] [PATCH proxmox-apt 2/5] repo: make suite_variant helper more general Fabian Ebner
2021-07-29 12:25 ` [pve-devel] [PATCH proxmox-apt 3/5] check repos: have caller specify the current suite Fabian Ebner
2021-07-29 12:25 ` [pve-devel] [PATCH proxmox-apt 4/5] repo: remove has_suite_variant helper Fabian Ebner
2021-07-29 12:25 ` [pve-devel] [PATCH proxmox-apt 5/5] add type DebianCodename Fabian Ebner
2021-07-29 12:25 ` [pve-devel] [PATCH pve-rs 1/2] apt: repos: adapt to back-end changes Fabian Ebner
2021-07-29 12:25 ` [pve-devel] [PATCH pve-rs 2/2] apt: repos: adapt to further " Fabian Ebner
2021-07-30  8:47 ` [pve-devel] applied-series: Re: [PATCH-SERIES proxmox-apt/pve-rs] better detection of standard repositories Thomas Lamprecht

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20210729122554.148980-1-f.ebner@proxmox.com \
    --to=f.ebner@proxmox.com \
    --cc=pve-devel@lists.proxmox.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.
Service provided by Proxmox Server Solutions GmbH | Privacy | Legal