Bug 2405006 - Review Request: copyparty - Portable file server with many supported protocols
Summary: Review Request: copyparty - Portable file server with many supported protocols
Keywords:
Status: NEW
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Jelle van der Waa
QA Contact: Fedora Extras Quality Assurance
URL: https://github.com/9001/copyparty
Whiteboard:
Depends On: 2406787
Blocks:
TreeView+ depends on / blocked
 
Reported: 2025-10-19 16:52 UTC by Simon de Vlieger
Modified: 2025-11-04 19:57 UTC (History)
3 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed:
Type: ---
Embargoed:
jvanderwaa: fedora-review?


Attachments (Terms of Use)

Description Simon de Vlieger 2025-10-19 16:52:57 UTC
Spec URL: https://raw.githubusercontent.com/supakeen/fedora-copyparty/99172a420ab76b29f87eafbf89b276690ec153bd/copyparty.spec
SRPM URL: https://github.com/supakeen/fedora-copyparty/blob/99172a420ab76b29f87eafbf89b276690ec153bd/copyparty-1.19.16-5.fc44.src.rpm
Description: Portable file server with many supported protocols
Fedora Account System Username: supakeen

Comment 1 Simon de Vlieger 2025-10-19 16:57:47 UTC
Note that this package includes vendored javascript, there's an elaborate comment in the spec file on the why/how. Upstream does the following things with javascript: https://github.com/9001/copyparty/blob/hovudstraum/scripts/deps-docker/Dockerfile#L25-L172 many of these are not packaged in Fedora and while the original files (pre-patch) could probably be included in the release tarball and then patched inside the RPM to me this feels like a large extra burden to keep that in sync with upstream.

Separately, I left out any user setup and/or service files in this initial package. I thought it over a bit and didn't really have a good idea on how to set this up in a way that would satisfy all users. If there's input on this I'd be happy to hear. Right now the idea is that users can install the package and run `copyparty` directly in a directly they want to share, potentially adding a service file themselves.

Comment 2 Fedora Review Service 2025-10-20 00:50:01 UTC
Cannot find any valid SRPM URL for this ticket. Common causes are:

- You didn't specify `SRPM URL: ...` in the ticket description
  or any of your comments
- The URL schema isn't HTTP or HTTPS
- The SRPM package linked in your URL doesn't match the package name specified
  in the ticket summary


---
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 Ben Beasley 2025-10-20 10:57:18 UTC
(In reply to Fedora Review Service from comment #2)
> Cannot find any valid SRPM URL for this ticket.

If you’re going to host your spec and SRPM on GitHub, please provide the “raw” SRPM link rather than linking to the web UI. The following should work:

Spec URL: https://raw.githubusercontent.com/supakeen/fedora-copyparty/99172a420ab76b29f87eafbf89b276690ec153bd/copyparty.spec
SRPM URL: https://github.com/supakeen/fedora-copyparty/raw/99172a420ab76b29f87eafbf89b276690ec153bd/copyparty-1.19.16-5.fc44.src.rpm

Comment 4 Simon de Vlieger 2025-10-20 11:07:15 UTC
Thanks Ben, my bad; I did use the `raw` URL for the spec file but copy/pasted the wrong one for the SRPM. Is your comment enough for `fedora-review-service` to retrigger?

Comment 5 Ben Beasley 2025-10-20 11:08:39 UTC
(In reply to Simon de Vlieger from comment #4)
> Thanks Ben, my bad; I did use the `raw` URL for the spec file but
> copy/pasted the wrong one for the SRPM. Is your comment enough for
> `fedora-review-service` to retrigger?

It should have been, but I don’t see any evidence of it yet. Just in case:

[fedora-review-service-build]

Comment 6 Simon de Vlieger 2025-10-21 07:35:39 UTC
(In reply to Ben Beasley from comment #5)
> It should have been, but I don’t see any evidence of it yet. Just in case:

Since there were some outages yesterday that might've affected things let's try once more:

Spec URL: https://raw.githubusercontent.com/supakeen/fedora-copyparty/99172a420ab76b29f87eafbf89b276690ec153bd/copyparty.spec
SRPM URL: https://raw.githubusercontent.com/supakeen/fedora-copyparty/99172a420ab76b29f87eafbf89b276690ec153bd/copyparty-1.19.16-5.fc44.src.rpm

[fedora-review-service-build]

Comment 7 Fedora Review Service 2025-10-21 07:41:23 UTC
Copr build:
https://copr.fedorainfracloud.org/coprs/build/9709472
(succeeded)

Review template:
https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2405006-copyparty/fedora-rawhide-x86_64/09709472-copyparty/fedora-review/review.txt

Found issues:

- License file COPYING.txt is not marked as %license
  Read more: https://docs.fedoraproject.org/en-US/packaging-guidelines/LicensingGuidelines/#_license_text

Please know that there can be false-positives.

---
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 8 Simon de Vlieger 2025-10-28 13:02:27 UTC
Since I devendored upstream I need `python-dnslib` available; I've filed an unretirement request and the related BZ for re-review is: https://bugzilla.redhat.com/show_bug.cgi?id=2406787

Comment 9 Jelle van der Waa 2025-11-04 19:57:05 UTC
Few notable things:

* two trivial lint issues

copyparty-u2c.noarch: W: summary-not-capitalized upload2copyparty client application.
copyparty-u2c.noarch: W: summary-ended-with-dot upload2copyparty client application.

* - copyparty.noarch: E: non-executable-script /usr/lib/python3.14/site-packages/copyparty/web/a/partyfuse.py 644 /usr/bin/env python3
* - copyparty.noarch: E: non-executable-script /usr/lib/python3.14/site-packages/copyparty/web/a/partyfuse.py 644 /usr/bin/env python3
* - copyparty.noarch: E: non-executable-script /usr/lib/python3.14/site-packages/copyparty/web/a/u2c.py 644 /usr/bin/env python3

prisonparty seems to be missing or is that intentional? 

upstream issues:

* [ ] /usr/lib/python3.14/site-packages/setuptools/dist.py:759: SetuptoolsDeprecationWarning: License classifiers are deprecated.
* [ ] Please use a simple string containing a SPDX expression for `project.license`. You can also use `project.license-files`. (Both options available on setuptools>=77.0.0).
* [ ] no man page, could be reported upstream (./scripts/help2txt.sh exists already)

The initial package lacks systemd support, so that makes deploying copyparty for a bit harder. I am not sure how useful a "copyparty" user is and a "copyparty.service" but I can see something like copyparty@.service enabling it for a dedicated user. However the standard WorkingDirectory seems a bit weird being /var/lib/copyparty-jail.


Note You need to log in before you can comment on or make changes to this bug.