public inbox for pve-devel@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
Service provided by Proxmox Server Solutions GmbH | Privacy | Legal