Bug 223873

Summary: x11-fm/thunar: Update to 1.6.13
Product: Ports & Packages Reporter: Olivier Duchateau <duchateau.olivier>
Component: Individual Port(s)Assignee: Jochen Neumeister <joneum>
Status: Closed FIXED    
Severity: Affects Many People CC: joneum, madpilot
Priority: --- Keywords: patch-ready
Version: LatestFlags: madpilot: maintainer-feedback+
Hardware: Any   
OS: Any   
Attachments:
Description Flags
Patch to update x11-fm/thunar
none
Patch to update x11-fm/thunar none

Description Olivier Duchateau 2017-11-25 20:29:27 UTC
Created attachment 188281 [details]
Patch to update x11-fm/thunar

Update to 1.6.13 (it is still Gtk2 application)
Comment 1 Jochen Neumeister freebsd_committer freebsd_triage 2017-11-25 20:43:16 UTC
I added @madpilot.
Can we commit that? Or are there problems with https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=217510
Comment 2 Olivier Duchateau 2017-11-25 22:09:55 UTC
Created attachment 188283 [details]
Patch to update x11-fm/thunar

Update patch, because now it exists a devel release (1.7.x).
Comment 3 Guido Falsi freebsd_committer freebsd_triage 2017-11-25 22:55:43 UTC
(In reply to Olivier Duchateau from comment #2)
> Created attachment 188283 [details]
> Patch to update x11-fm/thunar
> 
> Update patch, because now it exists a devel release (1.7.x).

(In reply to Jochen Neumeister from comment #1)
> I added @madpilot.
> Can we commit that? Or are there problems with
> https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=217510

Hi,

Sorry but what's the connection between this and bug 217510? (it seems related to wordpress and PHP).


Apart from that I don't see a problem with this patch. (the newer one)

Since you took the bug I have no objection with you committing it, after proper testing.

I just noticed it and had no time to test it, I could do that tomorrow.
Comment 4 Jochen Neumeister freebsd_committer freebsd_triage 2017-11-25 23:14:34 UTC
(In reply to Guido Falsi from comment #3)
108/5000
My mistake, I meant https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=217946

I will test the patch
Comment 5 Guido Falsi freebsd_committer freebsd_triage 2017-11-25 23:18:35 UTC
(In reply to Jochen Neumeister from comment #4)
> (In reply to Guido Falsi from comment #3)
> 108/5000
> My mistake, I meant https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=217946
> 

There should be no problem with that.

I identified a few problems in devel/glib20 port, for which I posted a patch in bug 199872.

There is still an outstanding crash in the present thunar, for which I only have partial backtraces. I have been unable to reproduce that crash in a jail with debugging binaries for now.

There's a slight chance the new version actually fixes the problem...We will see.

> I will test the patch

Thanks!
Comment 6 Guido Falsi freebsd_committer freebsd_triage 2017-11-25 23:20:08 UTC
> I identified a few problems in devel/glib20 port, for which I posted a patch
> in bug 199872.

I meant, problems in glib20 which cause thunar to crash.
Comment 7 commit-hook freebsd_committer freebsd_triage 2017-11-27 14:45:15 UTC
A commit references this bug:

Author: joneum
Date: Mon Nov 27 14:45:04 UTC 2017
New revision: 454959
URL: https://svnweb.freebsd.org/changeset/ports/454959

Log:
  x11-fm/thunar: Update from 1.6.12 to 1.6.13

  Changelog:
   - Handle cases when file watch is not supported
   - Missing Trash Translation String
   - Delete Thunar symlink on uninstall
   - Handle g_mem_is_system_malloc deprecation
   - Prefer local (file://) URIs than native ones (e.g. smb://)
   - Translation updates: Amharic, Arabic, Asturian, Bangla, Basque, Belarusian,
     Bulgarian, Catalan, Chinese (China), Chinese (Hong Kong), Chinese (Taiwan),
     Croatian, Czech, Danish, Dutch, English (Australia), English (United Kingdom),
     Esperanto, Estonian, Finnish, French, Galician, German, Greek, Hebrew,
     Hungarian, Icelandic, Indonesian, Italian, Japanese, Kazakh, Korean, Latvian,
     Lithuanian, Malay, Norwegian Bokm?l, Norwegian Nynorsk, Occitan, Persian,
     Polish, Portuguese, Portuguese (Brazil), Punjabi, Romanian, Russian, Serbian,
     Slovak, Spanish, Swedish, Telugu, Thai, Turkish, Ukrainian, Urdu,
     Urdu (Pakistan), Uyghur, Vietnamese

  PR:		223873
  Submitted by:	duchateau.olivier@gmail.com
  Approved by:	madpilot (for @xfce), rene (mentor)
  Differential Revision:	https://reviews.freebsd.org/D13257

Changes:
  head/x11-fm/thunar/Makefile
  head/x11-fm/thunar/distinfo
Comment 8 Jochen Neumeister freebsd_committer freebsd_triage 2017-11-27 14:45:42 UTC
thanks, landed :-)