GNU bug report logs - #32137
[PATCH] gnu: Add luakit

Previous Next

Full log

View this message in rfc822 format

From: help-debbugs <at> gnu.org (GNU bug Tracking System)
To: Raphaël Mélotte
 <raphael.melotte <at> gmail.com>
Subject: bug#32137: closed (Re: [bug#32137] [PATCH] gnu: Add luakit)
Date: Tue, 17 Jul 2018 13:15:02 +0000

[Message part 1 (text/plain, inline)]

Your bug report

#32137: [PATCH] gnu: Add luakit

which was filed against the guix-patches package, has been closed.

The explanation is attached below, along with your original report.
If you require more details, please reply to 32137 <at> debbugs.gnu.org.

-- 
32137: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=32137
GNU Bug Tracking System
Contact help-debbugs <at> gnu.org with problems

[Message part 2 (message/rfc822, inline)]

From: ludo <at> gnu.org (Ludovic Courtès)
To: Raphaël Mélotte <raphael.melotte <at> gmail.com>
Cc: 32137-done <at> debbugs.gnu.org
Subject: Re: [bug#32137] [PATCH] gnu: Add luakit
Date: Tue, 17 Jul 2018 15:14:39 +0200

[Message part 3 (text/plain, inline)]

Hello Raphaël,

Raphaël Mélotte <raphael.melotte <at> gmail.com> skribis:

> This patch adds luakit, a lightweight web browser based on webkitgtk.
>
> NB: this is my first patch submission, I apologize in advance if I did
> something wrong.
> From 36bae98701a1c04de2820781dd8b7488cac61e30 Mon Sep 17 00:00:00 2001
> From: 43317 <43317 <at> etu.he2b.be>
> Date: Fri, 13 Jul 2018 15:28:41 +0200
> Subject: [PATCH] add luakit
>
> ---
>  gnu/packages/web-browsers.scm | 65 +++++++++++++++++++++++++++++++++++
>  1 file changed, 65 insertions(+)

Thanks for this first patch and welcome!  I made only minor
modifications (patch below):

  1. Add ‘file-name’ to the source to please ‘guix lint’.

  2. Return #t at the end of the ‘wrap’ phase.

  3. Tweak the synopses and description.

  4. Change license to ‘gpl3+’ because source file headers explicitly
     say “version 3 or any later version”.

I provided a commit log that matches our conventions too.

  https://git.savannah.gnu.org/cgit/guix.git/commit/?id=700987f72c870b9a8e3ed6cd538064e2c2b8f3ce

That’s it!

I tried running “luakit -c /dev/null” but it insists on having an rc
file that explicitly creates a window, right?

Thank you!

Ludo’.

[Message part 4 (text/x-patch, inline)]

diff --git a/gnu/packages/web-browsers.scm b/gnu/packages/web-browsers.scm
index 02f076bbe..56bfa6ab2 100644
--- a/gnu/packages/web-browsers.scm
+++ b/gnu/packages/web-browsers.scm
@@ -144,7 +144,8 @@ features including, tables, builtin image display, bookmarks, SSL and more.")
                                   ".tar.gz"))
               (sha256
                (base32
-                "0dwxhnq90whakgdg21lzcf03n2g1c7hqgliwhav8av5na5mqpn93"))))
+                "0dwxhnq90whakgdg21lzcf03n2g1c7hqgliwhav8av5na5mqpn93"))
+              (file-name (string-append name "-" version ".tar.gz"))))
     (inputs
      `(("lua-5.1", lua-5.1)
        ("gtk+" ,gtk+)
@@ -184,14 +185,16 @@ features including, tables, builtin image display, bookmarks, SSL and more.")
                     (gtk (assoc-ref inputs "gtk+"))
                     (gtk-share (string-append gtk "/share")))
                (wrap-program (string-append luakit "/bin/luakit")
-                 `("LUA_CPATH" prefix (,(string-append
-                                         lua5.1-filesystem "/lib/lua/5.1/?.so;;"))))))))))
-    (synopsis "Fast, light and simple browser based on webkit")
-    (description "Luakit is a fast, light and simple to use micro-browser
- framework extensible by Lua
- using the WebKit web content engine and the GTK+ toolkit.")
+                 `("LUA_CPATH" prefix
+                   (,(string-append lua5.1-filesystem
+                                    "/lib/lua/5.1/?.so;;"))))
+               #t))))))
+    (synopsis "Fast, lightweight, and simple browser based on WebKit")
+    (description "Luakit is a fast, lightweight, and simple to use
+micro-browser framework extensible by Lua using the WebKit web content engine
+and the GTK+ toolkit.")
     (home-page "https://luakit.github.io/")
-    (license license:gpl3)))
+    (license license:gpl3+)))
 
 (define-public lynx
   (package

[Message part 5 (message/rfc822, inline)]

From: Raphaël Mélotte <raphael.melotte <at> gmail.com>
To: guix-patches <at> gnu.org
Subject: [PATCH] gnu: Add luakit
Date: Fri, 13 Jul 2018 15:38:20 +0200

[Message part 6 (text/plain, inline)]

This patch adds luakit, a lightweight web browser based on webkitgtk.

NB: this is my first patch submission, I apologize in advance if I did
something wrong.

[Message part 7 (text/html, inline)]

[0001-add-luakit.patch (text/x-patch, attachment)]

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.