all lists on lists.proxmox.com
 help / color / mirror / Atom feed
From: "Fabian Grünbichler" <f.gruenbichler@proxmox.com>
To: Proxmox VE development discussion <pve-devel@lists.proxmox.com>
Subject: [pve-devel] applied: [PATCH zfsonlinux 00/16] sync buildsys with debian-upstream
Date: Fri, 18 Jun 2021 15:44:55 +0200	[thread overview]
Message-ID: <1624023811.ief68m73km.astroid@nora.none> (raw)
In-Reply-To: <20210617135223.23472-1-s.ivanov@proxmox.com>

except for the last three patches - as discussed off-list:
- symbols file should probably be bumped to 2.0.4 (discuss with Debian 
  people? also how to handle libs without stability guarantees going 
  forward)
- rules override_dh_installsystemd could benefit from a reworked commit 
  message and comment in rules file ;)

On June 17, 2021 3:52 pm, Stoiko Ivanov wrote:
> This patchset started out as what is now the last commit:
> trying not to uselessly restart the zfs-services (like
> zfs-volume-wait.service) upon upgrading, since it lead to quite a few
> scary looking (but harmless) journal entries.
> 
> since this was already fixed in upstream debian's packaging I tried
> to pull in most recent changes which have not yet made it into
> our packaging. This should reduce the number of differences between
> debian and our packages (those can lead to unpleasant surprises), and also
> simply keeping in sync in the future.
> 
> changing to dh compat level 12 led to the patch for separate debug packages
> (deb-helper bug, which probably very few, if any, people will notice, since
> the version in bullseye and buster-backports is not affected)
> 
> some commits can probably be squashed (12+13 copying the linitian-overrides
> and dropping unused ones, 14+15 adding symbols files and updateing to 2.0.4)
> 
> While I did check the commits, selectively compared the results with
> diffoscope, and run a few rough tests (ztest on a VM of mine) a quick
> sanity-check by someone else would be highly appreciated (e.g. it's my first
> run-in with symbols files)
> 
> 
> Stoiko Ivanov (16):
>   buildsys: unify '{' and '(' in Makefile
>   buildsys: add all packages to make variable and sort
>   Reflect libssl requirement of libzfs
>   Elaborate the functionality of ZED in long description. (Closes:
>     #979414)
>   buildsys: switch to automatic debug packages
>   Bump debhelper compat level to 12.
>   compat 12 automatically adds initramfs trigger
>   Bump Standards-Version to 4.5.1 (no changes)
>   Adjust zed.d symlink-preservation
>   Remove /etc/zfs/zed.d on purge
>   sync scrub and trim cronjobs with debian upstream
>   update lintian-overrides
>   drop unused lintian overrides
>   add symbol files for library packages
>   update libzpool symbols
>   do not restart zfs-volume-wait.service
> 
>  Makefile                                      |   40 +-
>  debian/compat                                 |    1 -
>  debian/control                                |   31 +-
>  debian/libnvpair3linux.symbols                |  236 ++
>  debian/libuutil3linux.symbols                 |  261 ++
>  debian/libzfs4linux.symbols                   |  688 ++++
>  debian/libzfsbootenv1linux.lintian-overrides  |    4 +-
>  debian/libzfsbootenv1linux.symbols            |   10 +
>  debian/libzpool4linux.symbols                 | 3310 +++++++++++++++++
>  debian/rules                                  |    6 +-
>  .../zfsutils-linux/usr/lib/zfs-linux/scrub    |   43 +-
>  .../zfsutils-linux/usr/lib/zfs-linux/trim     |   60 +
>  debian/zfs-initramfs.triggers                 |    1 -
>  debian/zfs-test.lintian-overrides             |   14 +-
>  debian/zfs-zed.lintian-overrides              |    4 +
>  debian/zfs-zed.postinst                       |    2 +-
>  debian/zfs-zed.postrm                         |   17 +
>  debian/zfs-zed.prerm                          |    4 +-
>  debian/zfsutils-linux.cron.d                  |    5 +-
>  debian/zfsutils-linux.lintian-overrides       |   13 +
>  20 files changed, 4698 insertions(+), 52 deletions(-)
>  delete mode 100644 debian/compat
>  create mode 100644 debian/libnvpair3linux.symbols
>  create mode 100644 debian/libuutil3linux.symbols
>  create mode 100644 debian/libzfs4linux.symbols
>  create mode 100644 debian/libzfsbootenv1linux.symbols
>  create mode 100644 debian/libzpool4linux.symbols
>  create mode 100755 debian/tree/zfsutils-linux/usr/lib/zfs-linux/trim
>  delete mode 100644 debian/zfs-initramfs.triggers
>  create mode 100644 debian/zfs-zed.lintian-overrides
>  create mode 100644 debian/zfs-zed.postrm
> 
> -- 
> 2.20.1
> 
> 
> 
> _______________________________________________
> pve-devel mailing list
> pve-devel@lists.proxmox.com
> https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel
> 
> 
> 




      parent reply	other threads:[~2021-06-18 13:45 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-06-17 13:52 [pve-devel] " Stoiko Ivanov
2021-06-17 13:52 ` [pve-devel] [PATCH zfsonlinux 01/16] buildsys: unify '{' and '(' in Makefile Stoiko Ivanov
2021-06-17 13:52 ` [pve-devel] [PATCH zfsonlinux 02/16] buildsys: add all packages to make variable and sort Stoiko Ivanov
2021-06-17 13:52 ` [pve-devel] [PATCH zfsonlinux 03/16] Reflect libssl requirement of libzfs Stoiko Ivanov
2021-06-17 13:52 ` [pve-devel] [PATCH zfsonlinux 04/16] Elaborate the functionality of ZED in long description. (Closes: #979414) Stoiko Ivanov
2021-06-17 13:52 ` [pve-devel] [PATCH zfsonlinux 05/16] buildsys: switch to automatic debug packages Stoiko Ivanov
2021-06-17 13:52 ` [pve-devel] [PATCH zfsonlinux 06/16] Bump debhelper compat level to 12 Stoiko Ivanov
2021-06-17 13:52 ` [pve-devel] [PATCH zfsonlinux 07/16] compat 12 automatically adds initramfs trigger Stoiko Ivanov
2021-06-17 13:52 ` [pve-devel] [PATCH zfsonlinux 08/16] Bump Standards-Version to 4.5.1 (no changes) Stoiko Ivanov
2021-06-17 13:52 ` [pve-devel] [PATCH zfsonlinux 09/16] Adjust zed.d symlink-preservation Stoiko Ivanov
2021-06-17 13:52 ` [pve-devel] [PATCH zfsonlinux 10/16] Remove /etc/zfs/zed.d on purge Stoiko Ivanov
2021-06-17 13:52 ` [pve-devel] [PATCH zfsonlinux 11/16] sync scrub and trim cronjobs with debian upstream Stoiko Ivanov
2021-06-17 13:52 ` [pve-devel] [PATCH zfsonlinux 12/16] update lintian-overrides Stoiko Ivanov
2021-06-17 13:52 ` [pve-devel] [PATCH zfsonlinux 13/16] drop unused lintian overrides Stoiko Ivanov
2021-06-17 13:52 ` [pve-devel] [PATCH zfsonlinux 14/16] add symbol files for library packages Stoiko Ivanov
2021-06-17 13:52 ` [pve-devel] [PATCH zfsonlinux 15/16] update libzpool symbols Stoiko Ivanov
2021-06-17 13:52 ` [pve-devel] [PATCH zfsonlinux 16/16] do not restart zfs-volume-wait.service Stoiko Ivanov
2021-06-18 13:44 ` Fabian Grünbichler [this message]

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=1624023811.ief68m73km.astroid@nora.none \
    --to=f.gruenbichler@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