Gtk4 port

Signed-off-by: Michael Hofmann <mhofmann@redhat.com>

Author: mh21

Makefile.am

@@ -4,7 +4,7 @@ AM_CPPFLAGS = $(TERMDEPS_CFLAGS)
 
 AM_CFLAGS = -Wall -Wno-discarded-qualifiers -Wno-pointer-sign -Wno-unused-but-set-variable -Wno-unused-variable
 
-VALAFLAGS = --pkg=gtk+-3.0 --pkg=vte-2.91 --pkg=pango --pkg=posix --enable-checking
+VALAFLAGS = --pkg=gtk4 --pkg=vte-2.91-gtk4 --pkg=pango --pkg=posix --enable-checking
 
 # the terminal
 

configure.ac

@@ -13,7 +13,7 @@ AM_SILENT_RULES([yes])
 AM_PROG_VALAC([0.14.0])
 AM_PROG_CC_C_O
 
-PKG_CHECK_MODULES(TERMDEPS, [gtk+-3.0 vte-2.91 >= 0.66.0])
+PKG_CHECK_MODULES(TERMDEPS, [gtk4 vte-2.91-gtk4 >= 0.66.0])
 
 AC_CONFIG_FILES([Makefile])
 

packit.yml

@@ -5,20 +5,22 @@ specfile_path: simple-term.spec
 upstream_package_name: simple-term
 downstream_package_name: simple-term
 
+.job_template: &job_template
+  job: copr_build
+  # vte291-gtk4 not available in FC36
+  targets: [fedora-37, fedora-38, fedora-rawhide]
+
 jobs:
-  - job: copr_build
+  - <<: *job_template
     trigger: pull_request
-    targets: [fedora-all]
 
-  - job: copr_build
+  - <<: *job_template
     trigger: commit
     branch: main
-    targets: [fedora-all]
     owner: mh21
     project: simple-term-latest
 
-  - job: copr_build
+  - <<: *job_template
     trigger: release
-    targets: [fedora-all]
     owner: mh21
     project: simple-term

simple-term.spec

@@ -10,8 +10,8 @@ Source0: https://github.com/mh21/simple-term/archive/refs/tags/v%{version}.tar.g
 
 BuildRequires: gnome-common
 BuildRequires: vala
-BuildRequires: gtk3-devel
-BuildRequires: vte291-devel
+BuildRequires: gtk4-devel
+BuildRequires: vte291-gtk4-devel
 
 %description
 Simple vte-based terminal

simple-term.vala

@@ -34,7 +34,7 @@ class TerminalWindow : Gtk.Window
     private const string link_expr = "(((file|http|ftp|https)://)|(www|ftp)[-A-Za-z0-9]*\\.)[-A-Za-z0-9\\.]+(:[0-9]*)?(/[-A-Za-z0-9_\\$\\.\\+\\!\\*\\(\\),;:@&=\\?/~\\#\\%]*[^]'\\.}>\\) ,\\\"])?";
     private int link_tag;
 
-    private const Gdk.ModifierType key_mask = Gdk.ModifierType.SHIFT_MASK | Gdk.ModifierType.CONTROL_MASK | Gdk.ModifierType.MOD1_MASK |
+    private const Gdk.ModifierType key_mask = Gdk.ModifierType.SHIFT_MASK | Gdk.ModifierType.CONTROL_MASK | Gdk.ModifierType.ALT_MASK |
                                               Gdk.ModifierType.SUPER_MASK | Gdk.ModifierType.HYPER_MASK | Gdk.ModifierType.META_MASK;
 
     public TerminalWindow(Gtk.Application app, string[] command, string? title,
@@ -46,15 +46,17 @@ class TerminalWindow : Gtk.Window
         this.font = font;
 
         terminal = new Vte.Terminal();
-        add(terminal);
+        set_child(terminal);
 
         terminal.child_exited.connect(() => { destroy(); });
         terminal.decrease_font_size.connect(decrease_font_size_cb);
         terminal.increase_font_size.connect(increase_font_size_cb);
         terminal.window_title_changed.connect(window_title_changed_cb);
+        /*
         terminal.key_press_event.connect(key_press_event_cb);
         terminal.button_press_event.connect(button_press_event_cb);
         terminal.drag_data_received.connect(drag_data_received_cb);
+        */
 
         terminal.set_audible_bell(false);
         terminal.set_cursor_blink_mode(Vte.CursorBlinkMode.SYSTEM);
@@ -69,8 +71,10 @@ class TerminalWindow : Gtk.Window
         if (font != null)
             terminal.set_font(Pango.FontDescription.from_string(this.font));
 
+        /*
         if (role != null)
             this.set_role(role);
+        */
 
         try {
             var regex = new Vte.Regex.for_match(link_expr, -1, PCRE2_MULTILINE);
@@ -81,9 +85,11 @@ class TerminalWindow : Gtk.Window
             // ignored
         }
 
+        /*
         Gtk.drag_dest_set(terminal, Gtk.DestDefaults.ALL, {}, Gdk.DragAction.COPY);
         Gtk.drag_dest_add_uri_targets(terminal); // prefer URIs to text
         Gtk.drag_dest_add_text_targets(terminal);
+        */
 
         try {
             terminal.spawn_sync(Vte.PtyFlags.DEFAULT,
@@ -99,10 +105,10 @@ class TerminalWindow : Gtk.Window
             Idle.add(() => { destroy(); return false; });
         }
 
-        show_all();
+        present();
     }
 
-    private static Gdk.RGBA? get_color(string str)
+    private static Gdk.RGBA? parse_color(string str)
     {
         var color = Gdk.RGBA();
         if (!color.parse(str)) {
@@ -143,6 +149,7 @@ class TerminalWindow : Gtk.Window
         }
     }
 
+    /*
     public override bool delete_event(Gdk.EventAny event)
     {
         var pty = terminal.get_pty();
@@ -165,6 +172,7 @@ class TerminalWindow : Gtk.Window
         dialog.destroy();
         return result != Gtk.ResponseType.ACCEPT;
     }
+    */
 
     private void decrease_font_size_cb()
     {
@@ -181,6 +189,7 @@ class TerminalWindow : Gtk.Window
         set_title(terminal.get_window_title());
     }
 
+    /*
     private bool button_press_event_cb(Gdk.EventButton event)
     {
         if (((event.state & key_mask) == Gdk.ModifierType.CONTROL_MASK) && (event.button == 1)) {
@@ -239,6 +248,7 @@ class TerminalWindow : Gtk.Window
         }
         Gtk.drag_finish(context, true, false, time);
     }
+    */
 
     public void update_colors(string fg, string bg, string palette)
     {
@@ -248,8 +258,8 @@ class TerminalWindow : Gtk.Window
 
         Gdk.RGBA[] colors = null;
         foreach (var color in palette.split(","))
-            colors += get_color(color);
-        terminal.set_colors(get_color(fg), get_color(bg), colors);
+            colors += parse_color(color);
+        terminal.set_colors(parse_color(fg), parse_color(bg), colors);
     }
 }