Dear mentors,

I am looking for a sponsor for my package libopenxlsx.

   * Package name    : libopenxlsx
   * Version         : 0.5.2
   * Upstream contact: Lars Uffmann <coding23@uffmann.name> (myself)
   * URL             : https://codeberg.org/lars_uffmann/OpenXLSX
   * License         : BSD-3-Clause
   * VCS             : https://salsa.debian.org/lars_uffmann/libopenxlsx

This is a library I have helped maintain (with a lot of effort) since
early 2024, and I feel it has a wide enough userbase that it would be a
helpful package in debian for some developers. We recently migrated from
github to codeberg, in case you wonder why there are so few followers
there. On github it has 1.8K stars.

This is my first time ever trying to package for debian, so I would
appreciate a thorough review of my submission to salsa, and I will
implement feedback as best as I can.

The library documentation is currently mainly in the README.md and in
the Examples - the Demo programs in the Examples folder give a good
overview of implemented functionality, and how to use it. I am actively
implementing new features but have temporarily focused on polishing the
upstream repository in preparation for debian packaging.

The Intent To Package can be found at
https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=1140161

I will be happy to answer questions.

Kind regards

    Lars

Control: tags -1 +moreinfo
Hi Lars,

here's the review of openxlsx:

*Lintian:*

Running lintian...
N:
I: libopenxlsx source: older-debian-watch-file-standard 4 [debian/watch]
N:
N:   The version= line in the debian/watch file in this package declares an
N:   older version. Please upgrade when you have a chance.
N:
N:   Please refer to the uscan(1) manual page for details.
N:
N:   Visibility: info
N:   Show-Always: no
N:   Check: debian/watch/standard
N:

I: Lintian run successful (worst tag: info)

*Comment:* Consider using Watch version 5. You can get a glimpse of what
it should look like here [1]


*Summary:*

Autopkgtest: no tests
Build Architecture: amd64
Build Type: full
Build-Space: 226240
Build-Time: 36
Distribution: unstable
Host Architecture: amd64
Install-Time: 49
Job: /home/aryan/Documents/libopenxlsx_0.5.2-1.dsc
Lintian: successful (worst tag: info)
Machine Architecture: amd64
Package: libopenxlsx
Package-Time: 98
Source-Version: 0.5.2-1
Space: 226240
Status: successful
Version: 0.5.2-1

*Additional Comments:*

  * Your original ITP bug mentions the package name as "openxlsx" but
    your RFS was filed as "libopenxlsx". You should change this line [2]
    to "openxlsx".
  * The priority line in d/control [3] can be removed as it's the
    default behaviour.
  * Adding Vcs-Git and VCS-Browse fields would be good. Refer [4].
  * You're missing a hashtag # before the bug ID here [5].

Again like I said, I'm not a DD yet so I cannot sponsor your package.
Hope you find a sponsor soon.

That said, I can notice that there was a lot of LLM usage for this
package. I'm sure your sponsor will have more inputs about that as I'm
seeing a lot of files that might not be necessary.

Remove the moreinfo tag once my comments are addressed.


[1]
https://salsa.debian.org/spaciouskarter78/laser/-/blob/master/debian/watch

[2]
https://salsa.debian.org/lars_uffmann/libopenxlsx/-/blob/debian/latest/debian/control?ref_type=heads#L1

[3]
https://salsa.debian.org/lars_uffmann/libopenxlsx/-/blob/debian/latest/debian/control?ref_type=heads#L3

[4]
https://salsa.debian.org/spaciouskarter78/laser/-/blob/master/debian/control?ref_type=heads#L31

[5]
https://salsa.debian.org/lars_uffmann/libopenxlsx/-/blob/debian/latest/debian/changelog?ref_type=heads#L3