|Summary:||games/0ad: adds missing USES flags, removes implicit dependency on csh|
|Product:||Ports & Packages||Reporter:||Lorenzo Salvadore <salvadore>|
|Component:||Individual Port(s)||Assignee:||Guido Falsi <madpilot>|
|Severity:||Affects Only Me||CC:||madpilot|
Description Lorenzo Salvadore 2019-04-26 17:08:52 UTC
Created attachment 204043 [details] 0ad - USES flags and csh patch 0ad uses gl, gnome and sdl: it should be declared in the USES variable. My patch adds the missing USES flags. The file files/patch-libraries_sources_fcollada_src_Makefile introduces a dependency on csh. It is true that csh is in the base system, however some users (such as myself) compile their base system with WITHOUT_TCSH=yes: those users does not have csh and can not build the port. My patch modifies the file with an sh-compatible solution, so that the dependency is not needed any more.
Comment 1 Guido Falsi 2019-04-26 20:03:59 UTC
(In reply to Lorenzo Salvadore from comment #0) > Created attachment 204043 [details] > 0ad - USES flags and csh patch > Thanks for the patch! > 0ad uses gl, gnome and sdl: it should be declared in the USES variable. My > patch adds the missing USES flags. Good catch! > > The file files/patch-libraries_sources_fcollada_src_Makefile introduces a > dependency on csh. It is true that csh is in the base system, however some > users (such as myself) compile their base system with WITHOUT_TCSH=yes: > those users does not have csh and can not build the port. My patch modifies > the file with an sh-compatible solution, so that the dependency is not > needed any more. I added that code to remove the dependency on bash, and csh provided an easy way to do that. As you observed it's in base so I thought it was a non issue. Anyway your patch looks ok at first glance, let me test it a little and I'll see to commit it.
Comment 2 commit-hook 2019-04-28 15:10:09 UTC
A commit references this bug: Author: madpilot Date: Sun Apr 28 15:09:44 UTC 2019 New revision: 500318 URL: https://svnweb.freebsd.org/changeset/ports/500318 Log: - Modify patch to not implicitly depend on csh during build. - Add some missing USES PR: 237584 Submitted by: Lorenzo Salvadore <email@example.com> Changes: head/games/0ad/Makefile head/games/0ad/files/patch-libraries_source_fcollada_src_Makefile
Comment 3 Guido Falsi 2019-04-28 15:10:35 UTC