IMHO, it would otherwise be a regression not to tweak.

Attached is the 4th revision to update tint2.  This patch restores the
post-install sed command that edits Launcher paths in
/etc/xdg/tint2/tint2rc and /usr/local/share/examples/tint2/*.tint2rc:
/usr/share/applicatons -> /usr/local/share/applications.  Builds/runs on
amd64.


On Aug 13, 2017: 13:16, Brian Callahan wrote:
Hi Scott --

On 08/10/17 21:03, Heppler, J. Scott wrote:

Jggimi reviewed the port and found an extra tab and line in the
Makefile.

Version 3 of the attached diff corrects.


This mostly looks good. There's some whitespace/newline tweaks that I can fix on commit.
Question though: I get a lot of errors that look like this:
Could not open file /usr/share/applications/tint2conf.desktop

Should these be tweaked?

It worked just fine here, was even smart enough to preserve my fluxbox bar settings as it took over which was a nice touch.

Also, do you want to be MAINTAINER?

~Brian

Thanks

Scott
On Aug 10, 2017: 10:23, Brian Callahan wrote:
Hi Scott --

I'll take a look at this later today.

~Brian

On 08/09/17 20:01, Heppler, J. Scott wrote:
A revised diff to update x11/tint2 is attached.  Upstream has added 2
new features:  1)  Configurable "Buttons" that attach an icon to a
command.  2) Shell script "Executors" with examples for Network
monitoring and disk usage.

Changelog here:
https://github.com/o9000/tint2

I cvs add'd/rm;d the revised patches and portchecked.

Testers and feedback appreciated.





--
J. Scott Heppler
Index: Makefile
===================================================================
RCS file: /cvs/ports/x11/tint2/Makefile,v
retrieving revision 1.5
diff -u -p -u -r1.5 Makefile
--- Makefile    26 Nov 2016 02:06:16 -0000      1.5
+++ Makefile    20 Aug 2017 01:31:48 -0000
@@ -2,15 +2,14 @@
 
 COMMENT=        freedesktop-compliant panel
 
-V=             0.12.2
-REVISION=      2
-CATEGORIES=    x11
-DISTNAME=      tint2-$V
-DISTFILES=     ${DISTNAME}${EXTRACT_SUFX}{archive${EXTRACT_SUFX}?ref=v$V}
-WRKDIST=       ${WRKDIR}/tint2-v$V-dc300c021531f47cca336e15b50adf3b4a9bc9ae
-INSTDIR=       ${PREFIX}/tint2
+V=              0.14.6
+CATEGORIES=     x11
+DISTNAME=       tint2-$V
+DISTFILES=      ${DISTNAME}${EXTRACT_SUFX}{archive${EXTRACT_SUFX}?ref=v$V}
+WRKDIST=        ${WRKDIR}/tint2-v$V-8046600a552774253939b8a83bdd3fc0e60dacec
+INSTDIR=        ${PREFIX}/tint2
 
-HOMEPAGE=      https://gitlab.com/o9000/tint2/
+HOMEPAGE=       https://gitlab.com/o9000/tint2/
 
 # GPLv2 only
 PERMIT_PACKAGE_CDROM= YES
@@ -24,31 +23,36 @@ WANTLIB += gobject-2.0 gthread-2.0 gtk-x
 WANTLIB += pangocairo-1.0 pangoft2-1.0 pthread rsvg-2 startup-notification-1
 WANTLIB += z
 
-BUILD_DEPENDS= devel/gettext-tools
-LIB_DEPENDS=   graphics/imlib2 \
-               devel/gettext \
-               devel/pango \
-               devel/startup-notification \
-               x11/gnome/librsvg \
-               x11/gtk+2
-RUN_DEPENDS=   x11/py-gtk2 \
-               devel/desktop-file-utils
-
-MODULES=       devel/cmake \
-               lang/python \
-               x11/gnome
+BUILD_DEPENDS=  devel/gettext-tools
 
-MODGNOME_TOOLS+= gtk-update-icon-cache
+LIB_DEPENDS=    devel/gettext \
+                devel/pango \
+                devel/startup-notification \
+                graphics/imlib2 \
+                x11/gnome/librsvg \
+                x11/gtk+2
 
-NO_TEST=       Yes
+
+MODULES=        devel/cmake \
+                x11/gnome
+
+MODGNOME_TOOLS+= desktop-file-utils \
+                gtk-update-icon-cache \
+                shared-mime-info
+
+NO_TEST=        Yes
 
 CONFIGURE_ARGS= -DMANDIR="${PREFIX}/man"
 
 post-install:
-       cp -R ${WRKSRC}/sample/ ${PREFIX}/share/examples/tint2/
-       sed -i -e '/\/usr\/local\/share\/applications/d' -e \
-           's,/usr/share/applications,${LOCALBASE}/share/applications,g' \
-           ${PREFIX}/share/examples/tint2/*tint2rc
-       rm ${PREFIX}/share/tint2/*.tint2rc
+        cp -R ${WRKSRC}/themes/ ${PREFIX}/share/examples/tint2/
+        sed -i -e '/\/usr\/local\/share\/applications/d' -e \
+            's,/usr/share/applications,${LOCALBASE}/share/applications,g' \
+            ${PREFIX}/share/examples/tint2/*tint2rc
+        rm ${PREFIX}/share/tint2/tint2rc
+        rm ${PREFIX}/share/tint2/*.tint2rc
+        rm ${PREFIX}/share/examples/tint2/CMakeLists*
 
 .include <bsd.port.mk>
+
+
Index: distinfo
===================================================================
RCS file: /cvs/ports/x11/tint2/distinfo,v
retrieving revision 1.1.1.1
diff -u -p -u -r1.1.1.1 distinfo
--- distinfo    11 Sep 2015 14:14:07 -0000      1.1.1.1
+++ distinfo    20 Aug 2017 01:31:48 -0000
@@ -1,2 +1,2 @@
-SHA256 (tint2-0.12.2.tar.gz) = vanMkT1Ln9QTKF6jRGasFJVZ6emTjhKfRqZkwxEBM0U=
-SIZE (tint2-0.12.2.tar.gz) = 229320
+SHA256 (tint2-0.14.6.tar.gz) = F3Dio33IBmozr3OFuhu+dAUQG87Y6fx+6V1IzvU801E=
+SIZE (tint2-0.14.6.tar.gz) = 466533
Index: patches/patch-CMakeLists_txt
===================================================================
RCS file: patches/patch-CMakeLists_txt
diff -N patches/patch-CMakeLists_txt
--- /dev/null   1 Jan 1970 00:00:00 -0000
+++ patches/patch-CMakeLists_txt        20 Aug 2017 01:31:48 -0000
@@ -0,0 +1,14 @@
+$OpenBSD$
+
+Index: CMakeLists.txt
+--- CMakeLists.txt.orig
++++ CMakeLists.txt
+@@ -257,7 +257,7 @@ set_target_properties( tint2 PROPERTIES LINK_FLAGS "-p
+ install( TARGETS tint2 DESTINATION bin )
+ install( FILES tint2.svg DESTINATION 
${CMAKE_INSTALL_DATADIR}/icons/hicolor/scalable/apps )
+ install( FILES tint2.desktop DESTINATION 
${CMAKE_INSTALL_DATADIR}/applications )
+-install( FILES themes/tint2rc DESTINATION /etc/xdg/tint2 )
++#install( FILES themes/tint2rc DESTINATION /etc/xdg/tint2 )
+ install( FILES default_icon.png DESTINATION ${CMAKE_INSTALL_DATADIR}/tint2 )
+ install( FILES AUTHORS ChangeLog README.md doc/tint2.md DESTINATION ${docdir} 
)
+ install( FILES doc/manual.html doc/readme.html DESTINATION ${htmldir} )
Index: patches/patch-src_tint2conf_CMakeLists_txt
===================================================================
RCS file: patches/patch-src_tint2conf_CMakeLists_txt
diff -N patches/patch-src_tint2conf_CMakeLists_txt
--- patches/patch-src_tint2conf_CMakeLists_txt  11 Sep 2015 14:14:07 -0000      
1.1.1.1
+++ /dev/null   1 Jan 1970 00:00:00 -0000
@@ -1,8 +0,0 @@
-$OpenBSD: patch-src_tint2conf_CMakeLists_txt,v 1.1.1.1 2015/09/11 14:14:07 
sthen Exp $
---- src/tint2conf/CMakeLists.txt.orig  Tue Aug 11 09:26:49 2015
-+++ src/tint2conf/CMakeLists.txt       Fri Sep 11 15:30:10 2015
-@@ -71,4 +71,3 @@ add_subdirectory(po)
- install( TARGETS tint2conf DESTINATION bin )
- install( FILES tint2conf.svg DESTINATION 
${DATADIR}/icons/hicolor/scalable/apps )
- install( FILES tint2conf.desktop DESTINATION ${DATADIR}/applications )
--install( CODE "execute_process(COMMAND gtk-update-icon-cache -f -t 
${DATADIR}/icons/hicolor WORKING_DIRECTORY ${CMAKE_INSTALL_PREFIX})" )
Index: patches/patch-src_tint2conf_main_c
===================================================================
RCS file: patches/patch-src_tint2conf_main_c
diff -N patches/patch-src_tint2conf_main_c
--- patches/patch-src_tint2conf_main_c  26 Nov 2016 02:06:16 -0000      1.1
+++ /dev/null   1 Jan 1970 00:00:00 -0000
@@ -1,12 +0,0 @@
-$OpenBSD: patch-src_tint2conf_main_c,v 1.1 2016/11/26 02:06:16 jca Exp $
---- src/tint2conf/main.c.orig  Tue Nov 22 21:42:54 2016
-+++ src/tint2conf/main.c       Tue Nov 22 21:43:58 2016
-@@ -503,7 +503,7 @@ static void set_current_theme()
- 
-               gchar *main_file = g_build_filename(g_get_user_config_dir(), 
"tint2", "tint2rc", NULL);
-               {
--                      gchar *backup_path = g_strdup_printf("%s.backup.%ld", 
main_file, time(NULL));
-+                      gchar *backup_path = g_strdup_printf("%s.backup.%lld", 
main_file, (long long)time(NULL));
-                       copy_file(main_file, backup_path);
-                       g_free(backup_path);
-               }
Index: patches/patch-src_tint2conf_properties_c
===================================================================
RCS file: patches/patch-src_tint2conf_properties_c
diff -N patches/patch-src_tint2conf_properties_c
--- patches/patch-src_tint2conf_properties_c    26 Nov 2016 02:06:16 -0000      
1.1
+++ /dev/null   1 Jan 1970 00:00:00 -0000
@@ -1,12 +0,0 @@
-$OpenBSD: patch-src_tint2conf_properties_c,v 1.1 2016/11/26 02:06:16 jca Exp $
---- src/tint2conf/properties.c.orig    Tue Nov 22 21:44:18 2016
-+++ src/tint2conf/properties.c Tue Nov 22 21:44:32 2016
-@@ -171,7 +171,7 @@ void applyClicked(GtkWidget *widget, gpointer data)
-       gchar *file = get_current_theme_file_name();
-       if (file) {
-               if (config_is_manual(file)) {
--                      gchar *backup_path = g_strdup_printf("%s.backup.%ld", 
file, time(NULL));
-+                      gchar *backup_path = g_strdup_printf("%s.backup.%lld", 
file, (long long)time(NULL));
-                       copy_file(file, backup_path);
-                       g_free(backup_path);
-               }
Index: patches/patch-themes_CMakeLists_txt
===================================================================
RCS file: patches/patch-themes_CMakeLists_txt
diff -N patches/patch-themes_CMakeLists_txt
--- /dev/null   1 Jan 1970 00:00:00 -0000
+++ patches/patch-themes_CMakeLists_txt 20 Aug 2017 01:31:48 -0000
@@ -0,0 +1,9 @@
+$OpenBSD$
+
+Index: themes/CMakeLists.txt
+--- themes/CMakeLists.txt.orig
++++ themes/CMakeLists.txt
+@@ -1,2 +1,2 @@
+-file( GLOB EXTRATHEMEFILES *.tint2rc )
++file( GLOB EXTRATHEMEFILES tint2rc *.tint2rc )
+ install( FILES ${EXTRATHEMEFILES} DESTINATION ${CMAKE_INSTALL_DATADIR}/tint2 )
Index: pkg/PLIST
===================================================================
RCS file: /cvs/ports/x11/tint2/pkg/PLIST,v
retrieving revision 1.1.1.1
diff -u -p -u -r1.1.1.1 PLIST
--- pkg/PLIST   11 Sep 2015 14:14:07 -0000      1.1.1.1
+++ pkg/PLIST   20 Aug 2017 01:31:48 -0000
@@ -1,4 +1,4 @@
-@comment $OpenBSD: PLIST,v 1.1.1.1 2015/09/11 14:14:07 sthen Exp $
+@comment $OpenBSD$
 @pkgpath x11/tint
 @bin bin/tint2
 @bin bin/tint2conf
@@ -9,8 +9,18 @@ share/doc/tint2/
 share/doc/tint2/AUTHORS
 share/doc/tint2/ChangeLog
 share/doc/tint2/README.md
+share/doc/tint2/html/
+share/doc/tint2/html/images/
+share/doc/tint2/html/images/panel_padding.jpg
+share/doc/tint2/html/images/panel_size_margin.jpg
+share/doc/tint2/html/images/task_padding.jpg
+share/doc/tint2/html/images/taskbar_padding.jpg
+share/doc/tint2/html/manual.html
+share/doc/tint2/html/readme.html
+share/doc/tint2/tint2.md
 share/examples/tint2/
 @sample ${SYSCONFDIR}/xdg/
+@sample ${SYSCONFDIR}/xdg/tint2/
 share/examples/tint2/horizontal-dark-opaque.tint2rc
 share/examples/tint2/horizontal-dark-transparent.tint2rc
 share/examples/tint2/horizontal-light-opaque.tint2rc
@@ -32,22 +42,26 @@ share/examples/tint2/text_only_4.tint2rc
 share/examples/tint2/text_only_5.tint2rc
 share/examples/tint2/text_only_6.tint2rc
 share/examples/tint2/tint2rc
-@sample ${SYSCONFDIR}/xdg/tint2/
 @sample ${SYSCONFDIR}/xdg/tint2/tint2rc
 share/examples/tint2/vertical-dark-opaque.tint2rc
 share/examples/tint2/vertical-dark-transparent.tint2rc
 share/examples/tint2/vertical-light-opaque.tint2rc
 share/examples/tint2/vertical-light-transparent.tint2rc
+share/examples/tint2/vertical-neutral-icons.tint2rc
 share/icons/hicolor/scalable/apps/tint2.svg
 share/icons/hicolor/scalable/apps/tint2conf.svg
 share/locale/bs/LC_MESSAGES/tint2conf.mo
 share/locale/fr/LC_MESSAGES/tint2conf.mo
 share/locale/hr/LC_MESSAGES/tint2conf.mo
 share/locale/pl/LC_MESSAGES/tint2conf.mo
+share/locale/ru/LC_MESSAGES/tint2conf.mo
 share/locale/sr/LC_MESSAGES/tint2conf.mo
+share/mime/packages/tint2conf.xml
 share/tint2/
 share/tint2/default_icon.png
 @exec %D/bin/update-desktop-database
 @unexec-delete %D/bin/update-desktop-database
 @exec %D/bin/gtk-update-icon-cache -q -t %D/share/icons/hicolor
 @unexec-delete %D/bin/gtk-update-icon-cache -q -t %D/share/icons/hicolor
+@exec %D/bin/update-mime-database %D/share/mime
+@unexec-delete %D/bin/update-mime-database %D/share/mime

Reply via email to