Bug 2262536

Summary: Review Request: sway-contrib - Collection of user-contributed scripts for Sway
Product: [Fedora] Fedora Reporter: Aleksei Bavshin <alebastr89>
Component: Package ReviewAssignee: Jiri Konecny <jkonecny>
Status: CLOSED ERRATA QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: rawhideCC: jkonecny, package-review
Target Milestone: ---Keywords: AutomationTriaged
Target Release: ---Flags: jkonecny: fedora-review+
Hardware: Unspecified   
OS: Unspecified   
URL: https://github.com/OctopusET/sway-contrib
Whiteboard:
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2024-02-04 23:24:23 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description Aleksei Bavshin 2024-02-03 18:20:20 UTC
Spec URL: https://alebastr.fedorapeople.org/sway-contrib.spec
SRPM URL: https://alebastr.fedorapeople.org/sway-contrib-1.9~rc2-1.fc39.src.rpm

Description:
Collection of user-contributed scripts for Sway.

Fedora Account System Username: alebastr

Comment 1 Aleksei Bavshin 2024-02-03 18:20:22 UTC
This package built on koji:  https://koji.fedoraproject.org/koji/taskinfo?taskID=112883602

Comment 2 Fedora Review Service 2024-02-03 19:44:51 UTC
Copr build:
https://copr.fedorainfracloud.org/coprs/build/6983940
(succeeded)

Review template:
https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2262536-sway-contrib/fedora-rawhide-x86_64/06983940-sway-contrib/fedora-review/review.txt

Please take a look if any issues were found.


---
This comment was created by the fedora-review-service
https://github.com/FrostyX/fedora-review-service

If you want to trigger a new Copr build, add a comment containing new
Spec and SRPM URLs or [fedora-review-service-build] string.

Comment 3 Jiri Konecny 2024-02-04 22:22:53 UTC
Would be great to resolve these two RPM build issues:

*** WARNING: ./usr/libexec/sway-contrib/firefox-focus-monitor.py is executable but has no shebang, removing executable bit
mangling shebang in /usr/bin/grimshot from /bin/sh to #!/usr/bin/sh

Comment 4 Aleksei Bavshin 2024-02-04 23:05:52 UTC
Spec URL: https://alebastr.fedorapeople.org/sway-contrib.spec
SRPM URL: https://alebastr.fedorapeople.org/sway-contrib-1.9~rc2-1.fc39.src.rpm

> *** WARNING: ./usr/libexec/sway-contrib/firefox-focus-monitor.py is executable but has no shebang, removing executable bit

Fixed by preserving the permissions of the scripts

    drwxr-xr-x    2 root     root                        0 Feb  3 16:00 /usr/libexec/sway-contrib
    -rwxr-xr-x    1 root     root                     3533 Jan 20 08:58 /usr/libexec/sway-contrib/autoname-workspaces.py
    -rw-r--r--    1 root     root                     4048 Jan 20 08:58 /usr/libexec/sway-contrib/firefox-focus-monitor.py
    -rwxr-xr-x    1 root     root                     2014 Jan 20 08:58 /usr/libexec/sway-contrib/inactive-windows-transparency.py
    -rwxr-xr-x    1 root     root                     1527 Jan 20 08:58 /usr/libexec/sway-contrib/layout-per-window.py
    -rwxr-xr-x    1 root     root                     3129 Jan 20 08:58 /usr/libexec/sway-contrib/switch-top-level.py
    drwxr-xr-x    2 root     root                        0 Feb  3 16:00 /usr/share/licenses/sway-contrib
    -rw-r--r--    1 root     root                     1057 Jan 20 08:58 /usr/share/licenses/sway-contrib/LICENSE

I'll try to send a fix upstream before the final release.

> mangling shebang in /usr/bin/grimshot from /bin/sh to #!/usr/bin/sh

That's not an error, /usr/bin/sh is specific to (most of) systemd distributions and will be incorrect on other systems:

    <alebastr@freebsd-dev-64:~>
    zsh/2 288 % ls /usr/bin/sh
    ls: /usr/bin/sh: No such file or directory

Comment 5 Jiri Konecny 2024-02-04 23:11:31 UTC
I agree with the explanation. Thank you!

I don't see any other issue. ACKing this review.

Comment 6 Fedora Admin user for bugzilla script actions 2024-02-04 23:12:35 UTC
The Pagure repository was created at https://src.fedoraproject.org/rpms/sway-contrib

Comment 7 Fedora Update System 2024-02-04 23:23:15 UTC
FEDORA-2024-030bda6025 has been submitted as an update to Fedora 40. https://bodhi.fedoraproject.org/updates/FEDORA-2024-030bda6025

Comment 8 Fedora Update System 2024-02-04 23:24:23 UTC
FEDORA-2024-030bda6025 has been pushed to the Fedora 40 stable repository.
If problem still persists, please make note of it in this bug report.