[go: up one dir, main page]

Menu

Git Merge Request #2: Add Debian packaging (rejected)

Merging...

Merged

Something went wrong. Please, merge manually

Checking if merge is possible...

Something went wrong. Please, merge manually

Oliver Smith wants to merge 12 commits from /u/osmith/sispmctl/ to master, 2023-07-04

Import the debian packaging from https://deb.debian.org/debian/pool/main/s/sispmctl/sispmctl_3.1-3.debian.tar.xz and modernize it for 4.10.

I've uploaded a new source package with these changes to mentors.debian.net, looks like it is still processing.

Commit Date  
[e2e844] (upstream-debian-packaging) by Oliver Smith Oliver Smith

debian/changelog: add 4.10

Signed-off-by: Oliver Smith <osmith@...>

2023-06-28 16:01:57 Tree
[2f8245] by Oliver Smith Oliver Smith

debian/control: change priority extra -> optional

Fix lintian warning:
W: sispmctl: priority-extra-is-replaced-by-priority-optional

Signed-off-by: Oliver Smith <osmith@...>

2023-06-28 15:56:12 Tree
[a6bb56] by Oliver Smith Oliver Smith

debian: install udev rule and create related group

postinst script based on the example here:
https://www.debian.org/doc/manuals/securing-debian-manual/bpp-lower-privs.en.html

Signed-off-by: Oliver Smith <osmith@...>

2023-06-28 15:03:54 Tree
[30a7ec] by Oliver Smith Oliver Smith

debian: split libsispmctl0 into subpackage

Fix lintian warnings:
W: sispmctl: link-to-shared-library-in-wrong-package usr/lib/x86_64-linux-gnu/libsispmctl.so.0.2.1 usr/lib/x86_64-linux-gnu/libsispmctl.so
W: sispmctl: package-name-doesnt-match-sonames libsispmctl0

Signed-off-by: Oliver Smith <osmith@...>

2023-06-28 13:52:36 Tree
[067c47] by Oliver Smith Oliver Smith

debian/rules: use --disable-static

The static library causes lintian errors, remove it for now.

E: sispmctl: incorrect-libdir-in-la-file usr/lib/x86_64-linux-gnu/libsispmctl.la (line 41) usr/lib/ != usr/lib/x86_64-linux-gnu/
E: sispmctl: non-empty-dependency_libs-in-la-file usr/lib/x86_64-linux-gnu/libsispmctl.la (line 20) -lusb

Signed-off-by: Oliver Smith <osmith@...>

2023-06-28 13:36:56 Tree
[f015ae] by Oliver Smith Oliver Smith

debian/control: update build-depends

* set libusb-dev min version to 0.1.11, as in configure.ac
* add pkg-config to fix:
syntax error near unexpected token `LIBUSB,'

Signed-off-by: Oliver Smith <osmith@...>

2023-06-28 13:26:42 Tree
[049a30] by Oliver Smith Oliver Smith

gitignore: add files created during deb build

Signed-off-by: Oliver Smith <osmith@...>

2023-06-28 13:19:02 Tree
[0773b7] by Oliver Smith Oliver Smith

debian: bump compat level to 10

Signed-off-by: Oliver Smith <osmith@...>

2023-06-28 13:22:43 Tree
[6d6e49] by Oliver Smith Oliver Smith

debian: remove cdbs dependency

Build without cdbs (Common Debian Build System) as it has been orphaned.

Signed-off-by: Oliver Smith <osmith@...>

2023-06-28 13:03:00 Tree
[e5bd53] by Oliver Smith Oliver Smith

debian/source/format: set to native

Signed-off-by: Oliver Smith <osmith@...>

2023-06-28 12:51:51 Tree
[74f9e1] by Oliver Smith Oliver Smith

debian/patches: remove

The patch in this directory was a backport, it is included in 4.10.

Signed-off-by: Oliver Smith <osmith@...>

2023-06-28 13:32:14 Tree
[5058ef] by Oliver Smith Oliver Smith

debian: import 3.1-3 packaging

Import packaging that is currently in debian:
https://deb.debian.org/debian/pool/main/s/sispmctl/sispmctl_3.1-3.debian.tar.xz

Signed-off-by: Oliver Smith <osmith@...>

2023-06-28 12:48:34 Tree

Discussion

  • xypron

    xypron - 2023-06-30

    Hello Oliver,

    thank you for looking into Debian packaging. The best place to host Debian packaging is https://salsa.debian.org/. This is what Debian maintainers are using.

    Please, have a look at this branch https://github.com/xypron/sispmctl/tree/debian. You merge request does not include setting up udev rules and adding a user group.

    Best regards

    Heinrich

     
  • Oliver Smith

    Oliver Smith - 2023-06-30

    Thanks for the quick reply!

    The best place to host Debian packaging is https://salsa.debian.org/. This is what Debian maintainers are using.

    So you prefer not having it be part of the main repository? I'm not sure what's the best practice here, some projects appear to have the debian directory upstream.

    I can push the patches to salsa.debian.org too.

    Please, have a look at this branch https://github.com/xypron/sispmctl/tree/debian. You merge request does not include setting up udev rules and adding a user group.

    Thanks, I was not aware of this branch. I'll take a look and compare. But the merge request does include setting up the udev rules and the user group, see this patch:

    https://sourceforge.net/u/osmith/sispmctl/ci/a6bb560ad337a8b38f91b8ff97189ec4255db909/

    And in your commits I've seen that you have a canonical mail address. Are you a Debian developer or do you know somebody who could "sponsor" the package to get it updated in Debian?

    PS: Out of curiosity, any reason this project is still on sourceforge?

    Best regards,
    Oliver

     
  • Oliver Smith

    Oliver Smith - 2023-07-04
    • Status: open --> rejected
     

Log in to post a comment.