Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 25 Jul 2018 16:32:34 +0000
From:      bugzilla-noreply@freebsd.org
To:        gnome@FreeBSD.org
Subject:   maintainer-approval requested: [Bug 214338] [PATCH] devel/glib20: new kqueue() backend for file monitoring : [Attachment 195450] Proposed patch (since 473551 revision)
Message-ID:  <bug-214338-6497-BzGoJQkx5m@https.bugs.freebsd.org/bugzilla/>
In-Reply-To: <bug-214338-6497@https.bugs.freebsd.org/bugzilla/>
References:  <bug-214338-6497@https.bugs.freebsd.org/bugzilla/>

next in thread | previous in thread | raw e-mail | index | archive | help
lightside <lightside@gmx.com> has asked freebsd-gnome mailing list
<gnome@FreeBSD.org> for maintainer-approval:
Bug 214338: [PATCH] devel/glib20: new kqueue() backend for file monitoring
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=3D214338

Attachment 195450: Proposed patch (since 473551 revision)
https://bugs.freebsd.org/bugzilla/attachment.cgi?id=3D195450&action=3Dedit



--- Comment #45 from lightside <lightside@gmx.com> ---
Created attachment 195450
  --> https://bugs.freebsd.org/bugzilla/attachment.cgi?id=3D195450&action=
=3Dedit
Proposed patch (since 473551 revision)

Hello.

I tested build for patch in attachment #191448 after ports r473551 changes =
and
found following linker error on FreeBSD 10.4 amd64:
-8<--
./.libs/libgio-2.0.so: undefined reference to `mstosbt'
cc: error: linker command failed with exit code 1 (use -v to see invocation)
-->8-

Therefore, I added implementation for mstosbt function to files/kqueue_fnm.c
file, based on:
https://svnweb.freebsd.org/base?view=3Drevision&revision=3D321686
https://svnweb.freebsd.org/base/head/sys/sys/time.h?view=3Dmarkup&pathrev=
=3D321686#
l198
https://svnweb.freebsd.org/base/head/sys/sys/param.h?revision=3D321688&view=
=3Dmarku
p#l61

The `make check-plist` command found some errors (excerpt):
-8<--
=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: lib/charset.alias
Error: Orphaned: share/doc/gio/GAction.html
<..>
Error: Orphaned: share/doc/gobject/up.png
=3D=3D=3D> Checking for items in pkg-plist which are not in STAGEDIR
=3D=3D=3D> Error: Plist issues found.
*** Error code 1

Stop.
<..>
-->8-

This is because of autoreconf usage on pre-configure-FAM_ALTBACKEND-on stag=
e,
which overwrites previous changes for Makefile.in files (see bug #226920,
comment #4 for another explanation). I fixed this with using some sed chang=
es
for ${WRKSRC}/gio/kqueue/Makefile.in file. The
files/extra-patch-gio_kqueue_Makefile.am patch was removed.

Attached modified patch.

Probably, there is a need to clarify description for FAM_ALTBACKEND option.=
 For
example, usage of "Alternative" instead of "Alternate" word.



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?bug-214338-6497-BzGoJQkx5m>