Sunday, August 27, 2017

Re: Revised x11/tint2 0.12.2 -> 0.14.6 update

Summary so far:

1) Updated 0.12.2 - 0.14.6
2) Removed rc files with Linux /usr/share/application path
3) Patched default tint2rc with /usr/local/share/application path
4) Searched tint2conf/main.c tint2conf/properties.c for incorrect
declaration and patched to print_t %lld (long long)

Current Issues:

1) diff with 2 additional patchs to correct print_t %lld (long long)
seems incomplete - missing new line numbers. How to get the new @@ line
numbers to show in the diff?
2) I'm still also getting the portcheck message regarding WANTLIB look
liked masked RUN_DEPENDS and did not understand the previous guidance to
add back x11/gtk+2 to LIB_DEPENDS - I had moved it but it is in
LIB_DEPENDS currently
3) Port Module x11/gnome - If x11/gnome is used should I specify Configure Style
= cmake?

patch-src_tint2conf_main_c:

$OpenBSD$

Index: src/tint2conf/main.c
--- src/tint2conf/main.c.orig
+++ src/tint2conf/main.c
@@ -100,7 +100,7 @@ char *file_name_from_path(const char *filepath)

void make_backup(const char *filepath)
{
- gchar *backup_path = g_strdup_printf("%s.backup.%ld", filepath, time(NULL));
+ gchar *backup_path = g_strdup_printf("%s.backup.%lld", filepath, (long long)time(NULL));
copy_file(filepath, backup_path);
g_free(backup_path);
}

and

patch-src_tint2conf_properties_c:

$OpenBSD$

Index: src/tint2conf/properties.c
--- src/tint2conf/properties.c.orig
+++ src/tint2conf/properties.c
@@ -180,7 +180,7 @@ void applyClicked(GtkWidget *widget, gpointer data)
gchar *filepath = get_current_theme_path();
if (filepath) {
if (config_is_manual(filepath)) {
- gchar *backup_path = g_strdup_printf("%s.backup.%ld", filepath, time(NULL));
+ gchar *backup_path = g_strdup_printf("%s.backup.%lld", filepath, (long long)time(NULL));
copy_file(filepath, backup_path);
g_free(backup_path);
}



--
J. Scott Heppler

No comments:

Post a Comment