french/verbiste update to 0.1.41
AbandonedPublic
Actions

Authored by linimon on Nov 26 2014, 6:38 AM.

Details

Reviewers

naddy
dereckson

Summary

Port updated to 0.1.41
GUI is now enabled by default in upstream configure file
Adding NLS knob for gettext
Tidying pkg-plist

Test Plan

Poudriere, see P38 for results.

Diff Detail

Repository

rP FreeBSD ports repository

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

dereckson updated this revision to Diff 2552.Nov 26 2014, 6:38 AM

dereckson retitled this revision from to french/verbiste update to 0.1.41.

dereckson updated this object.

dereckson edited the test plan for this revision. (Show Details)

Relevant Bugzilla bug: https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=195402

naddy added a subscriber: naddy.Nov 26 2014, 10:12 PM

naddy added inline comments.

Makefile
30	This can't be right. If NLS is off, how do you ensure that verbiste will not link against libintl? You need --disable-nls. I'm not sure if this is possible (or makes sense) for the GTK client.
36–37	Has the GNOME option been tested at all? I think the last time time we broached this, you said you didn't use it and neither do I. I doubt that this option is correct after bit-rotting for years, and it should just be removed.

dereckson planned changes to this revision.Dec 6 2014, 12:18 PM

dereckson added a subscriber: kwm.

dereckson added inline comments.

Makefile
30	Okay, I rollback this one.
36–37	I've checked with @kwm, who didn't see any reason it couldn't be working I've then reinstalled a GUI environment, it still compiles and runs.
pkg-plist
25–31	This one has been replaced by by-size icons, so pkg-plist must be updated for GUI.

Restoring former gettext configuration
Fixing PNG icon files

dereckson edited the test plan for this revision. (Show Details)Dec 6 2014, 1:00 PM

dereckson edited the test plan for this revision. (Show Details)

I tried diff 2 on 10.1-STABLE/amd64, but neither the command-line tools nor the GTK client work at all. They simply abort with "conversion from wide characters to UTF-8 not supported".

wideToUTF8Conv = iconv_open("UTF-8", "WCHAR_T");

Does this require GNU iconv?

Can't repro on FreeBSD 10.

According man 3 iconv_open, it's included in Standard C Library.

So issue will be about WCHAR_T it seems.

I don't know why you can't reproduce the problem. It also affects the version in the ports tree (0.1.33_2) and the official packages are broken.

Converting to WCHAR_T is an extension and requires GNU iconv. This can simply be accomplished with USES=iconv:wchar_t. There is also a fix to src/verbiste/Makefile.in required to actually link libverbiste with libiconv.

I made a complete patch, but I can't figure out how to attach it to this revision in Differential. (An abortive attempt ended up as D1399.)

This revision now requires changes to proceed.Dec 30 2014, 9:14 PM