Skip to content

ci: fix pcsc run dir #18754

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 1 commit into from
Sep 1, 2025
Merged

ci: fix pcsc run dir #18754

merged 1 commit into from
Sep 1, 2025

Conversation

@siddarthkay
Copy link
Contributor

Summary

fixes path to ipcdir for pcscd, to solve errors like these:

$ ./StatusIm-Desktop-2.35.0-rc.4-e8083a-x86_64.AppImage 
starting pcscd with /tmp/.mount_StatusMlBcBD/usr/bin/pcscd -f &
00000000 [138400321718208] ../src/pcscdaemon.c:629:main() cannot create /pcscd/run: No such file or directory
qt.core.qobject.connect: QObject::connect(QObject, Unknown): invalid nullptr parameter

fixes: #18635

@siddarthkay siddarthkay self-assigned this Sep 1, 2025
@siddarthkay siddarthkay requested a review from a team as a code owner September 1, 2025 14:02
@siddarthkay siddarthkay linked an issue Sep 1, 2025 that may be closed by this pull request
Keycard reader not found/recognized in release Linux builds #18635
Open
@status-im-auto
Copy link
Member

status-im-auto commented Sep 1, 2025 •
edited
Loading

Jenkins Builds

❔ Commit #️⃣ Finished (UTC) Duration Platform Result
✔️ 892e0b1 #1 2025-09-01 14:10:41 ~8 min tests/nim 📄log
✔️ 892e0b1 #1 2025-09-01 14:16:32 ~13 min tests/ui 📄log
✔️ 892e0b1 #1 2025-09-01 14:19:59 ~17 min linux/x86_64 📦tgz
✔️ 892e0b1 #1 2025-09-01 14:27:20 ~24 min linux/x86_64-nwaku 📦tgz
✔️ 892e0b1 #1 2025-09-01 14:28:27 ~25 min macos/aarch64 🍎dmg
✔️ 892e0b1 pr18754 2025-09-01 14:31:07 ~11 min tests/e2e 📊rpt
✔️ 892e0b1 #1 2025-09-01 14:31:22 ~28 min windows/x86_64 💿exe
✔️ 892e0b1 #2 2025-09-01 15:16:37 ~18 min macos/aarch64-nwaku 🍎dmg

caybro
caybro approved these changes Sep 1, 2025
View reviewed changes
Copy link
Member

@caybro caybro left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Can we have this backported to the release branch pls?

@siddarthkay
Copy link
Contributor Author

@caybro : Can you or someone test the linux build and see if you don't get this keycard error please?

@siddarthkay siddarthkay merged commit 892e0b1 into master Sep 1, 2025
9 checks passed
@siddarthkay siddarthkay deleted the fix-pcscd-packaging-linux branch September 1, 2025 16:08
@siddarthkay
Copy link
Contributor Author

verified by @anastasiyaig

IMG_0007
IMG_0008

@siddarthkay siddarthkay mentioned this pull request Sep 1, 2025
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@alaibe alaibe alaibe approved these changes

@caybro caybro caybro approved these changes

Assignees

@siddarthkay siddarthkay

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Keycard reader not found/recognized in release Linux builds

4 participants

@siddarthkay @status-im-auto @alaibe @caybro