Hi Xiao,

Preamble...

Thanks for taking time to create this package and your contribution to Debian.

The below review is for assistance. It is offered to help submitters of
packages to Debian mentors improve their packages prior to possible
sponsorship into Debian. There is no obligation on behalf of the subitter to
make any alterations based upon information provided in the review.

Review...

1. Build: Good

2. Lintian: Issue

I: mintstick source: debian-watch-uses-insecure-uri 
http://packages.linuxmint.com/pool/main/m/mintstick/ [debian/watch:4]
N: 
N:   The watch file uses an unencrypted transport protocol for the URI such as
N:   http:// or ftp://. It is recommended to use a secure transport such as
N:   HTTPS for anonymous read-only access.
N:   
N:   Upstream may already provide a HTTPS variant of the URI. If not, please
N:   contact them and ask them to consider adding one.
N: 
N:   Visibility: info
N:   Show-Always: no
N:   Check: debian/watch
N:

Would be nice to get the release archive from the secure github project.

I: mintstick: desktop-entry-lacks-keywords-entry 
[usr/share/applications/mintstick-format-
kde.desktop]
N: 
N:   This .desktop file is either missing a Keywords entry, or it does not
N:   contain keywords above and beyond those already present in the Name or
N:   GenericName entries.
N:   
N:   The Keywords field is intended to show keywords relevant for a .desktop
N:   file.
N:   
N:   Desktop files are organized in key-value pairs and are similar to INI
N:   files.
N:   
N:   The desktop-file-validate tool in the desktop-file-utils package may be
N:   useful when checking the syntax of desktop entries.
N: 
N:   Please refer to
N:   
https://specifications.freedesktop.org/desktop-entry-spec/latest/ar01s06.html,
N:   Bug#693918, and
N:   https://wiki.gnome.org/Initiatives/GnomeGoals/DesktopFileKeywords for
N:   details.
N: 
N:   Visibility: info
N:   Show-Always: no
N:   Check: menu-format
N: 
N:
I: mintstick: desktop-entry-lacks-keywords-entry 
[usr/share/applications/mintstick-format.desktop]
N:
I: mintstick: desktop-entry-lacks-keywords-entry 
[usr/share/applications/mintstick-kde.desktop]
N:
I: mintstick: desktop-entry-lacks-keywords-entry 
[usr/share/applications/mintstick.desktop]

Can be requested upstream. They may wish to add them or not.

3. Licenses: Good

4. Build Twice (sudo pbuilder build --twice <package>.dsc): Good

5. Reproducible builds (reporotest)[1]: Good

6. Install (No previous installs): Good

7. Upgrade (Over previous installs if any): Good

Additional....

A. Why there is 'TERM=xterm' in scripts. One for - is it needed upstream etc. 
etc.? :-)

B. Not related, but would be nice to address. 'mintstick' is currently broken 
in Ubuntu 24.04 (maybe
other releases also - not tested) due to not installing dep 'python3-unidecode'.

Summary...

I believe mintstick is ready for sponsorship/upload. Could a Debian Developer 
(DD) with available
free time, please review this package and upload if you feel it is ready.

I hope the new maintainer will look at the issues and work with upstream to 
improve the package for
end users over future uploads.

[1] https://wiki.debian.org/ReproducibleBuilds/Howto#Newer_method

Regards

Phil

-- 

Internet Relay Chat (IRC): kathenas

Website: https://kathenas.org

Instagram: https://instagram.com/kathenasorg/

Buy Me A Coffee: https://buymeacoffee.com/kathenasorg

Attachment: signature.asc
Description: This is a digitally signed message part

Reply via email to