Bug 269595 - security/nitrokey-app: plist and stage-qa issues
Summary: security/nitrokey-app: plist and stage-qa issues
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: Fernando Apesteguía
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2023-02-16 19:56 UTC by Marcin Cieślak
Modified: 2023-02-18 17:37 UTC (History)
2 users (show)

See Also:
monwarez: maintainer-feedback+


Attachments
fixup staging issue (1.97 KB, patch)
2023-02-16 20:42 UTC, Thibault Payet
monwarez: maintainer-approval+
Details | Diff
fixup staging issue (1.92 KB, patch)
2023-02-17 10:08 UTC, Thibault Payet
monwarez: maintainer-approval+
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Marcin Cieślak 2023-02-16 19:56:41 UTC
PORTNAME=       nitrokey-app
DISTVERSIONPREFIX=      v
DISTVERSION=    1.4.2


root@radziecki:/usr/ports/security/nitrokey-app # make stage-qa
====> Running Q/A tests (stage-qa)
Error: /usr/local/bin/nitrokey-app is linked to /usr/local/lib/qt5/libQt5Core.so.5 from devel/qt5-core but it is not declared as a dependency
Warning: you need USES=qt:5 and USE_QT+=core
/!\ To run stage-qa automatically add DEVELOPER=yes to your environment /!\
root@radziecki:/usr/ports/security/nitrokey-app # make stage-qa
Makefile   distinfo   pkg-descr  pkg-plist  
root@radziecki:/usr/ports/security/nitrokey-app # make stage-qa
root@radziecki:/usr/ports/security/nitrokey-app # make check-plist
====> Checking for pkg-plist issues (check-plist)
===> Parsing plist
===> Checking for items in STAGEDIR missing from pkg-plist
Error: Orphaned: local/share/bash-completion/completions/nitrokey-app
===> Checking for items in pkg-plist which are not in STAGEDIR
Error: Missing: etc/bash_completion.d/nitrokey-app
===> Error: Plist issues found.
*** Error code 1

The latter prevents clean install
Comment 1 Thibault Payet 2023-02-16 20:36:41 UTC
Thanks for the report, I will upload a patch for this issue
Comment 2 Thibault Payet 2023-02-16 20:42:33 UTC
Created attachment 240189 [details]
fixup staging issue

Since this port install a .desktop files, I added desktop-file-utils
Comment 3 Marcin Cieślak 2023-02-17 09:19:08 UTC
Thank you for picking up so quickly!

It it better, but still not fully fixed. Here is the result of "make stage-qa check-plist" with the patch applied:

====> Running Q/A tests (stage-qa)
Warning: you may not need USES=desktop-file-utils
/!\ To run stage-qa automatically add DEVELOPER=yes to your environment /!\
====> Checking for pkg-plist issues (check-plist)
===> Parsing plist
===> Checking for items in STAGEDIR missing from pkg-plist
Error: Orphaned: local/share/bash-completion/completions/nitrokey-app
===> Checking for items in pkg-plist which are not in STAGEDIR
===> Error: Plist issues found.
*** Error code 1

Stop.
make: stopped in /usr/ports/security/nitrokey-app
root@radziecki:/usr/ports/security/nitrokey-app # find `make -V STAGEDIR` -name nitrokey-app
/big/ports/usr/ports/security/nitrokey-app/work/stage/usr/local/bin/nitrokey-app
/big/ports/usr/ports/security/nitrokey-app/work/stage/usr/local/share/bash-completion/completions/nitrokey-app
/big/ports/usr/ports/security/nitrokey-app/work/stage/usr/local/local/share/bash-completion/completions/nitrokey-app

Looks like additional copy of the completions gets installed in /usr/local/local/share ...
Comment 4 Thibault Payet 2023-02-17 09:37:51 UTC
(In reply to Marcin Cieślak from comment #3)
My fixes was to install it to the correct places by copying it to the correct places.

I need to find why it tries so hard to install it with a duplicate local prefix.

My guess is that it assumes default to be /usr/, so when it detect a localbase it append local. But in our case this gives /usr/local/local
Comment 5 Thibault Payet 2023-02-17 10:08:03 UTC
Created attachment 240205 [details]
fixup staging issue

They detect the bash completion directory with the command
pkg-config --variable=completionsdir bash-completion

and then remove the beginning with usr/

Since they provide a variable for it, it is easier to pass the correct one.
Comment 6 Marcin Cieślak 2023-02-18 15:31:22 UTC
This time:

root@radziecki:/usr/ports/security/nitrokey-app # make clean stage-qa check-plist

(...)

====> Running Q/A tests (stage-qa)
Warning: you may not need USES=desktop-file-utils
/!\ To run stage-qa automatically add DEVELOPER=yes to your environment /!\
====> Checking for pkg-plist issues (check-plist)
===> Parsing plist
===> Checking for items in STAGEDIR missing from pkg-plist
Error: Orphaned: local/share/bash-completion/completions/nitrokey-app
===> Checking for items in pkg-plist which are not in STAGEDIR
===> Error: Plist issues found.
*** Error code 1

Stop.
make: stopped in /usr/ports/security/nitrokey-app
root@radziecki:/usr/ports/security/nitrokey-app # find `make -V STAGEDIR` -name nitrokey-app
/big/ports/usr/ports/security/nitrokey-app/work/stage/usr/local/bin/nitrokey-app
/big/ports/usr/ports/security/nitrokey-app/work/stage/usr/local/share/bash-completion/completions/nitrokey-app
/big/ports/usr/ports/security/nitrokey-app/work/stage/usr/local/local/share/bash-completion/completions/nitrokey-app
Comment 7 Fernando Apesteguía freebsd_committer freebsd_triage 2023-02-18 17:36:20 UTC
Committed,

Thanks!
Comment 8 commit-hook freebsd_committer freebsd_triage 2023-02-18 17:37:06 UTC
A commit in branch main references this bug:

URL: https://cgit.FreeBSD.org/ports/commit/?id=44dda6cd41a7c640cae50d66e486a028608dea36

commit 44dda6cd41a7c640cae50d66e486a028608dea36
Author:     Thibault Payet <monwarez@mailoo.org>
AuthorDate: 2023-02-18 17:08:03 +0000
Commit:     Fernando Apesteguía <fernape@FreeBSD.org>
CommitDate: 2023-02-18 17:31:45 +0000

    security/nitrokey-app: Fix packaging

    PR:             269595
    Reported by:    saper@saper.info (maintainer)

 security/nitrokey-app/Makefile  | 8 ++++++--
 security/nitrokey-app/pkg-plist | 2 +-
 2 files changed, 7 insertions(+), 3 deletions(-)