Bug 237584
| 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> | ||||
| Status: | Closed FIXED | ||||||
| Severity: | Affects Only Me | CC: | madpilot | ||||
| Priority: | --- | Flags: | madpilot:
maintainer-feedback+
|
||||
| Version: | Latest | ||||||
| Hardware: | Any | ||||||
| OS: | Any | ||||||
| Attachments: |
|
||||||
(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. 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 <phascolarctos@protonmail.ch> Changes: head/games/0ad/Makefile head/games/0ad/files/patch-libraries_source_fcollada_src_Makefile Committed. Thanks! |
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.