Skip site navigation (1)Skip section navigation (2)
Date:      Sun, 06 Nov 2016 00:44:49 +0000
From:      bugzilla-noreply@freebsd.org
To:        freebsd-ports-bugs@FreeBSD.org
Subject:   [Bug 211855] emulators/higan: Update to 101
Message-ID:  <bug-211855-13-gzpfh7XTQ3@https.bugs.freebsd.org/bugzilla/>
In-Reply-To: <bug-211855-13@https.bugs.freebsd.org/bugzilla/>
References:  <bug-211855-13@https.bugs.freebsd.org/bugzilla/>

next in thread | previous in thread | raw e-mail | index | archive | help
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=3D211855

Jan Beich (mail not working) <jbeich@FreeBSD.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |jbeich@FreeBSD.org

--- Comment #5 from Jan Beich (mail not working) <jbeich@FreeBSD.org> ---
Comment on attachment 173826
  --> https://bugs.freebsd.org/bugzilla/attachment.cgi?id=3D173826
higan-101.patch

Can you provide a link to upstream changelog for this version that can be u=
sed
in commit message?

poudriere bulk -t complains

  =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D
  =3D=3D=3D=3D> Running Q/A tests (stage-qa)
  =3D=3D=3D=3D> Checking for pkg-plist issues (check-plist)
  =3D=3D=3D> Parsing plist
  =3D=3D=3D> Checking for items in STAGEDIR missing from pkg-plist
  Error: Orphaned: %%DATADIR%%/Mega Drive.sys/manifest.bml
  =3D=3D=3D> Checking for items in pkg-plist which are not in STAGEDIR
  =3D=3D=3D> Error: Plist issues found.
  *** [check-plist] Error code 1

On 11.0 or later build is broken:

  /usr/bin/ld: cannot find -lomp
  c++: error: linker command failed with exit code 1 (use -v to see invocat=
ion)

USES=3Dcompiler:openmp cannot be used here due to bug 199603 and bug 210337=
, so
either drop -fopenmp or make it conditional. Note, devel/openmp is currently
broken on FreeBSD i386 unlike Linux i386 or Windows 32bit.

> +MASTER_SITES=3D	http://download.byuu.org/ \

Does it still work? The homepage points at http://helmet.kafuka.org/

  fetch: http://download.byuu.org/higan_v101-source.7z: Not Found

> +USES=3D		7z:p7zip ...

Nothing to change here! libarchive on 10.* has trouble with solid archives =
but
works fine on 11.0+ or if compressed with -ms=3Doff. If more such consumers
appear it maybe worth to limit libarchive to 11.0+ in USES=3D7z.

  =3D> SHA256 Checksum OK for higan_v101-source.7z.
  tar: Damaged 7-Zip archive
  tar: Error exit delayed from previous errors.
  *** Error code 1

> +BROKEN_FreeBSD_9=3D	no C++14 support

It builds fine with lang/gcc5. Try replacing with

  FAVORITE_COMPILER=3D    ${COMPILER_TYPE} # c++14-lib

but as LDFLAGS aren't respected runtime fails

  $ higan
  /usr/lib/libgomp.so.1: version GOMP_4.0 required by /usr/local/bin/higan =
not
found

one way to fix is by passing default flags (see DO_MAKE_BUILD below)

  MAKE_ENV=3D     link=3D"${LDFLAGS} ${LIBS}"
  ...
  post-patch:
        ${REINPLACE_CMD} '/link :=3D/d' ${WRKSRC}/nall/GNUmakefile

>  MAKE_ENV+=3D	compiler=3D"${CXX}" cflags=3D"-x c -std=3Dc99 ${CFLAGS}" \
> -		cppflags=3D"-x c++ -std=3Dc++11 ${CXXFLAGS}"
> +		cppflags=3D"-x c++ -std=3Dc++14 ${CXXFLAGS}"

Convert -std=3D* to USE_CSTD=3Dc99 + USE_CXXSTD=3Dc++14.

> GTK2_DESC=3D		Build with GTK+2 GUI
> QT4_DESC=3D		Build with QT4 GUI
> GLX_DESC=3D		Build GLX video driver

Some of these should be dropped. See Mk/bsd.options.desc.mk

> +GTK2_USE=3D	GNOME=3Dcairo,gdkpixbuf2,gtk20,gtksourceview2
> +GTK2_USES=3D	gettext
> +GTK2_LIB_DEPENDS=3D	libfontconfig.so:x11-fonts/fontconfig \
> +			libfreetype.so:print/freetype2

Maybe sort these like you do for mandatory dependencies i.e., LIB_DEPENDS
before any USE*.
Also, some dependencies can be dropped via LDFLAGS +=3D -Wl,--as-needed

  --- higan     before
  +++ higan     after
  @@ -9,18 +9,11 @@ Dynamic section at offset 0x3c22f8 conta
    0x0000000000000001 (NEEDED)             Shared library:
[libgtksourceview-2.0.so.0]
    0x0000000000000001 (NEEDED)             Shared library:
[libgtk-x11-2.0.so.0]
    0x0000000000000001 (NEEDED)             Shared library:
[libgdk-x11-2.0.so.0]
  - 0x0000000000000001 (NEEDED)             Shared library:
[libpangocairo-1.0.so.0]
  - 0x0000000000000001 (NEEDED)             Shared library: [libatk-1.0.so.=
0]
    0x0000000000000001 (NEEDED)             Shared library: [libcairo.so.2]
    0x0000000000000001 (NEEDED)             Shared library:
[libgdk_pixbuf-2.0.so.0]
  - 0x0000000000000001 (NEEDED)             Shared library: [libgio-2.0.so.=
0]
  - 0x0000000000000001 (NEEDED)             Shared library:
[libpangoft2-1.0.so.0]
    0x0000000000000001 (NEEDED)             Shared library: [libpango-1.0.s=
o.0]
    0x0000000000000001 (NEEDED)             Shared library:
[libgobject-2.0.so.0]
    0x0000000000000001 (NEEDED)             Shared library: [libglib-2.0.so=
.0]
  - 0x0000000000000001 (NEEDED)             Shared library: [libintl.so.8]
  - 0x0000000000000001 (NEEDED)             Shared library:
[libfontconfig.so.1]
  - 0x0000000000000001 (NEEDED)             Shared library: [libfreetype.so=
.6]
    0x0000000000000001 (NEEDED)             Shared library: [libomp.so.0]
    0x0000000000000001 (NEEDED)             Shared library: [libc++.so.1]
    0x0000000000000001 (NEEDED)             Shared library: [libcxxrt.so.1]

>  do-build:
> -.for d in . ananke
> +.for d in higan icarus
>  	(cd ${WRKSRC}/${d} && ${MAKE_CMD} ${MAKE_ENV})
>  .endfor

Call DO_MAKE_BUILD instead of MAKE_CMD. MAKE_ENV isn't supposed to override
vendor variables unless ?=3D is used.

  MAKEFILE=3D     GNUmakefile
  ...
  do-build:
  .for d in higan icarus
        @${DO_MAKE_BUILD} ${ALL_TARGET} -C${WRKSRC}/${d}
  .endfor

> +	${MKDIR} ${STAGEDIR}${DATADIR}
> +	${INSTALL_DATA} ${WRKSRC}/higan/data/cheats.bml ${STAGEDIR}${DATADIR}/
> +	(cd ${WRKSRC}/higan/systems && ${COPYTREE_SHARE} \* ${STAGEDIR}${DATADI=
R}/)

MKDIR call can be omitted if you put COPYTREE_* before INSTALL_DATA.
Also, replace \* with . (dot).

--=20
You are receiving this mail because:
You are the assignee for the bug.=



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?bug-211855-13-gzpfh7XTQ3>