|Summary:||[patch] sysutils/xfce4-kbdleds-plugin make fetchable|
|Product:||Ports & Packages||Reporter:||Zane C. Bowers-Hadley <vvelox>|
|Component:||Individual Port(s)||Assignee:||freebsd-xfce (Nobody) <xfce>|
|Severity:||Affects Some People||CC:||duchateau.olivier, madpilot, vvelox|
Description Zane C. Bowers-Hadley 2019-12-25 11:59:57 UTC
Created attachment 210221 [details] svn diff Not finding the original tar.bz2 archive. Looking the git.xfce.org, I found it there as .tar.gz file. Although this patch is not check via the site list. Went and added a additional line there that makes this fetchable as a .tar.gz file.
Comment 1 Olivier Duchateau 2019-12-25 14:20:38 UTC
This panel is very outdated, no update for 8 years.
Comment 2 Guido Falsi 2019-12-25 21:13:21 UTC
Hi, The distfile you propose looks quite different from the original one, and while it comes from first hand source it needs some checking too not introduce differences. Apart from this, as Olivier noted, this plugin is quite old and looks unmaintained. While this is not a big problem right now, it will not work with XFCE 4.15 (development version) and XFCE 4.16 when that will be released without updates. This is due to XFCE 4.15 migrating to GTK3 only. So while it can stay in the tree for now, it will most probably have to be removed when XFCE will be updated to 4.16 in the future. I'll have a look and see if this new distfile is ok.
Comment 3 Olivier Duchateau 2019-12-25 21:46:11 UTC
When you add Git snapshot in MASTER_SITES, you need to adjust dependencies too, especially here: > BUILD_DEPENDS= xdt-autogen:devel/xfce4-dev-tools > ... > USE= autoreconf Because configure script has not yet generated.
Comment 4 Guido Falsi 2019-12-25 23:20:57 UTC
(In reply to Olivier Duchateau from comment #3) I'm testing his patch (with slight modifications, like changing master sites only inside the port) and it looks like the tarball actually includes a working configure script, so no need to add dependencies or regenerate it. BTW I noticed some linux distributions using this same distfile (same sha256 sum), this lessens my worries about unwanted differences.
Comment 5 commit-hook 2019-12-26 19:43:48 UTC
A commit references this bug: Author: madpilot Date: Thu Dec 26 19:43:22 UTC 2019 New revision: 520947 URL: https://svnweb.freebsd.org/changeset/ports/520947 Log: Make xfce4-kbdleds-plugin fetchable again by grabbing distfile from upstream git repo directly, remove BROKEN line. PR: 242879 Submitted by: Zane C. Bowers-Hadley <firstname.lastname@example.org> Changes: head/sysutils/xfce4-kbdleds-plugin/Makefile head/sysutils/xfce4-kbdleds-plugin/distinfo
Comment 6 Guido Falsi 2019-12-26 19:45:50 UTC
Committed, but made the MASTER_SITES change local to this port only. Thanks!