Bug 209301 - [PATCH] www/xombrero: Enable pararell building / Fix location of icons
Summary: [PATCH] www/xombrero: Enable pararell building / Fix location of icons
Status: Closed Feedback Timeout
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: Eygene Ryabinkin
URL:
Keywords: patch
Depends on:
Blocks:
 
Reported: 2016-05-05 18:35 UTC by tkato432
Modified: 2018-09-03 13:53 UTC (History)
2 users (show)

See Also:


Attachments
www_xombrero.diff (8.58 KB, patch)
2016-05-05 18:35 UTC, tkato432
no flags Details | Diff
www_xombrero.diff (3.28 KB, patch)
2016-05-06 10:20 UTC, tkato432
no flags Details | Diff
www_xombrero.diff (3.28 KB, patch)
2016-05-23 18:30 UTC, tkato432
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description tkato432 2016-05-05 18:35:37 UTC
Created attachment 170015 [details]
www_xombrero.diff
Comment 1 commit-hook freebsd_committer freebsd_triage 2016-05-05 21:45:24 UTC
A commit references this bug:

Author: rm
Date: Thu May  5 21:44:54 UTC 2016
New revision: 414668
URL: https://svnweb.freebsd.org/changeset/ports/414668

Log:
  www/xombrero: unbreak; update to 1.6.4

  This patch is cherry-picked from 4 different PRs and one upstream bug report [1]

  Changes are:
  - fix build and runtime (based on [1]). I also tried patches and solutions from
    PR's, but they didn't worked for some reason
  - update to 1.6.4
  - convert to options helpers and add other simplifications to Makefile
  - sync installed files with Makefile, bundled into distribution
  - add upstream patch to fix some GTK3 warnings

  This was tested both at build and runtime with both GTK versions.
  Also HTML5 video playback was tested - there was a report that it doesn't work.

  [1] https://github.com/conformal/xombrero/issues/131

  PR:		192779 203939 205646 209301
  Submitted by:	various persons

Changes:
  head/www/xombrero/Makefile
  head/www/xombrero/distinfo
  head/www/xombrero/files/patch-freebsd__Makefile
  head/www/xombrero/files/patch-xombrero.css
  head/www/xombrero/pkg-descr
  head/www/xombrero/pkg-plist
Comment 2 Ruslan Makhmatkhanov freebsd_committer freebsd_triage 2016-05-05 21:49:01 UTC
Update to 1.6.4 was just committed. Most of the changes from your patch were merged. Thanks.
Comment 3 Ruslan Makhmatkhanov freebsd_committer freebsd_triage 2016-05-05 21:49:56 UTC
Oh, Mathieu, sorry, my bad((
Comment 4 tkato432 2016-05-06 00:00:33 UTC
Well, desktop entry file and layout of icons in r414668 (and previous
versions) does not seem to follow freedesktop.org standard.
Comment 5 Ruslan Makhmatkhanov freebsd_committer freebsd_triage 2016-05-06 06:46:23 UTC
I see. However I'm sure this should be submitted upstream instead. Just do a pull request at https://github.com/conformal/xombrero. Upstream maintainer is quite responsive.
Comment 6 Ruslan Makhmatkhanov freebsd_committer freebsd_triage 2016-05-06 06:56:31 UTC
By the way, icon names are hardcoded in xombrero.c, so it may require to change that too.
Comment 7 tkato432 2016-05-06 10:20:11 UTC
Created attachment 170043 [details]
www_xombrero.diff

Hmm, it seems xombrero.c assumes PNG images are installed in $DATADIR.
Icons under $PREFIX/share/icons have to be made as symlinks from PNG
images in $DATADIR.

Workaround for pararell building is also implemented.
Comment 8 Ruslan Makhmatkhanov freebsd_committer freebsd_triage 2016-05-06 11:25:48 UTC
Please send this changes upstream. There is no point to maintain local patches when this could be just upstreamed and work out of the box. Users of other platforms will also benefit from this change. Thanks.

EXAMPLES option and conditional installation of examples will be committed soon. I missed that because no information in PR about what changes were introduced and why. 

PS. "GTK;WebBrower" looks like a typo to me.
Comment 9 tkato432 2016-05-06 23:30:33 UTC
Although Category field in xombrero.desktop might possibly be worth
informing, other things do not seem adequate to involve upstream.

At least, location of PNG images in existing port is an obvious bug.
The original $WRKSRC/freebsd/Makefile also intend to install images into
$DATADIR.
Comment 10 tkato432 2016-05-23 18:30:35 UTC
Created attachment 170577 [details]
www_xombrero.diff

Fix typo
Comment 11 tkato432 2016-05-23 18:33:34 UTC
Reopen
Comment 12 Ruslan Makhmatkhanov freebsd_committer freebsd_triage 2017-02-14 08:33:40 UTC
Reassign to current maintainer.
Comment 13 Walter Schwarzenfeld freebsd_triage 2018-01-23 00:57:00 UTC
What is the reason for still open?