Bug 189647 - [maintainer update] emulators/visualboyadvance-m to r1231
Summary: [maintainer update] emulators/visualboyadvance-m to r1231
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: Normal Affects Only Me
Assignee: John Marino
URL:
Keywords:
Depends on:
Blocks: 190695
  Show dependency treegraph
 
Reported: 2014-05-11 06:40 UTC by Allison Reid
Modified: 2014-06-20 10:22 UTC (History)
4 users (show)

See Also:


Attachments
file.diff (15.35 KB, patch)
2014-05-11 06:40 UTC, Allison Reid
no flags Details | Diff
Updated regenrated r1231 patch (15.73 KB, patch)
2014-06-15 21:57 UTC, Nicole Reid
no flags Details | Diff
VBAM r1231 update without LC_MESSAGES in plist (15.57 KB, patch)
2014-06-20 03:11 UTC, Nicole Reid
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Allison Reid 2014-05-11 06:40:00 UTC
- Disable broken SDL interface
- Add Link support
- Add Lirc support

Full commit log at http://sourceforge.net/p/vbam/code/1231/log/

Fix: Patch attached with submission follows:
Comment 1 Martin Wilke freebsd_committer freebsd_triage 2014-06-04 06:57:05 UTC
Hi,

Patching the makefile reject, could u please recreate a new patch against the current ports tree. thank you.
Comment 2 Nicole Reid 2014-06-15 21:57:36 UTC
Created attachment 143817 [details]
Updated regenrated r1231 patch

Since r1001:
- Add Link support
- Add Lirc support
- Patch SDL interface (thanks dennylin93@hs.ntnu.edu.tw)
- Several new translations
- Many small fixes for various titles

Full commit log at http://sourceforge.net/p/vbam/code/1231/log/
Comment 3 John Marino freebsd_committer freebsd_triage 2014-06-18 23:00:28 UTC
i think I promised to take this...
Comment 4 John Marino freebsd_committer freebsd_triage 2014-06-18 23:26:44 UTC
Hi Nicole, 
It is failing plist-sanity checks:
https://redports.org/buildarchive/20140618231835-77896/

It looks like a simple plist issue.  Can you provide a "delta" patch?  I assume only a patch to plist is needed.
Comment 5 Nicole Reid 2014-06-20 03:05:55 UTC
That's something I was mulling over actually. makeplist is giving me dirrmtrys for those two locale dirs:

Error: Orphaned: @dirrmtry share/locale/ca_ES/LC_MESSAGES
Error: Orphaned: @dirrmtry share/locale/ca_ES
Error: Orphaned: @dirrmtry share/locale/it_IT/LC_MESSAGES
Error: Orphaned: @dirrmtry share/locale/it_IT

But the build failure is exactly the opposite problem, those dirs specified in the plist but not on the filesystem:

pkg-static: lstat(/work/a/ports/emulators/visualboyadvance-m/work/stage/usr/local/share/locale/ca_ES/LC_MESSAGES/): No such file or directory
pkg-static: lstat(/work/a/ports/emulators/visualboyadvance-m/work/stage/usr/local/share/locale/ca_ES/): No such file or directory
pkg-static: lstat(/work/a/ports/emulators/visualboyadvance-m/work/stage/usr/local/share/locale/it_IT/LC_MESSAGES/): No such file or directory
pkg-static: lstat(/work/a/ports/emulators/visualboyadvance-m/work/stage/usr/local/share/locale/it_IT/): No such file or directory

I will attach the patch again with those four lines removed and see what redports thinks of it. This looks like something I should start using myself :)
Comment 6 Nicole Reid 2014-06-20 03:11:07 UTC
Created attachment 143947 [details]
VBAM r1231 update without LC_MESSAGES in plist

VBAM r1231 update without LC_MESSAGES in plist due to buildbot failure
Comment 7 John Marino freebsd_committer freebsd_triage 2014-06-20 10:03:13 UTC
If I were to guess on the original diff, it's because the file is install with the prefix "%%GVBAM%%%%NLS%" but the dirrmtry is prefixed only with "%%NLS%%" rather than "%%GVBAM%%%%NLS%".

I'll try adding that the dirrmtry line and see if that fixes things.
Comment 8 John Marino freebsd_committer freebsd_triage 2014-06-20 10:16:50 UTC
hypothesis confirmed, redports likes that.  I'll commit this shortly.
Comment 9 commit-hook freebsd_committer freebsd_triage 2014-06-20 10:20:41 UTC
A commit references this bug:

Author: marino
Date: Fri Jun 20 10:20:19 UTC 2014
New revision: 358546
URL: http://svnweb.freebsd.org/changeset/ports/358546

Log:
  emulators/visualboyadvance-m: Upgrade version r1001 => r1231

  Improvements:
    * Add Link support
    * Add Lirc support
    * Patch SDL interface (thanks dennylin93)
    * Several new translations
    * Many small fixes to various titles

  PR:		189647
  Submitted by:	maintainer (Nicole Reid)

Changes:
  head/emulators/visualboyadvance-m/Makefile
  head/emulators/visualboyadvance-m/distinfo
  head/emulators/visualboyadvance-m/files/patch-CMakeLists.txt
  head/emulators/visualboyadvance-m/files/patch-fex-fex-Data_Reader.cpp
  head/emulators/visualboyadvance-m/files/patch-src-Util.cpp
  head/emulators/visualboyadvance-m/files/patch-src-common-ffmpeg.cpp
  head/emulators/visualboyadvance-m/files/patch-src-gba-GBALink.cpp
  head/emulators/visualboyadvance-m/files/patch-src-gtk-filters.cpp
  head/emulators/visualboyadvance-m/files/patch-src-gtk-filters.h
  head/emulators/visualboyadvance-m/files/patch-src-gtk-gvbam.desktop
  head/emulators/visualboyadvance-m/files/patch-src-sdl-expr.cpp
  head/emulators/visualboyadvance-m/pkg-plist
Comment 10 John Marino freebsd_committer freebsd_triage 2014-06-20 10:22:22 UTC
Thanks for hanging in there.