all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Efraim Flashner <efraim@flashner.co.il>
To: Thierry <thierry@moncourriel.eu>
Cc: 38744@debbugs.gnu.org
Subject: [bug#38744] [PATCH] adding copyq 3.9.3 to xdisorg.scm
Date: Thu, 26 Dec 2019 08:56:54 +0200	[thread overview]
Message-ID: <20191226065654.GI23018@E5400> (raw)
In-Reply-To: <3ba0f830-f7d1-3301-a1a6-99a045735dd0@moncourriel.eu>

[-- Attachment #1: Type: text/plain, Size: 3500 bytes --]

Thank you for your patch! A couple of comments inline.

On Wed, Dec 25, 2019 at 10:07:24PM +0100, Thierry wrote:
> 

> From 3e5fb3f5fc852b0f512b6771c6950f4947f66dd7 Mon Sep 17 00:00:00 2001
> From: Grillon <thierry@moncourriel.eu>
> Date: Wed, 25 Dec 2019 21:22:42 +0100
> Subject: [PATCH] gnu: copyq(clipboard manager): added v3.9.3
> 
> * gnu/packages/xdisorg.scm (copyq): added from 3.9.3
> ---
>  gnu/packages/xdisorg.scm | 29 +++++++++++++++++++++++++++++
>  1 file changed, 29 insertions(+)
> 
> diff --git a/gnu/packages/xdisorg.scm b/gnu/packages/xdisorg.scm
> index b1759eabb2..9ee713de2f 100644
> --- a/gnu/packages/xdisorg.scm
> +++ b/gnu/packages/xdisorg.scm
> @@ -85,6 +85,7 @@
>    #:use-module (gnu packages guile)
>    #:use-module (gnu packages xml)
>    #:use-module (gnu packages gtk)
> +  #:use-module (gnu packages qt)
>    #:use-module (gnu packages xorg)
>    #:use-module (gnu packages fontutils)
>    #:use-module (gnu packages bison)
> @@ -229,6 +230,34 @@ with X11 or Wayland, or in a text terminal with ncurses.")
>      (license (list license:gpl3+        ; client program[s] and other sources
>                     license:lgpl3+))))   ; library and bindings
>  
> +(define-public copyq
> +(package
> +  (name "copyq")
> +  (version "3.9.3")
> +  (source (origin
> +            (method url-fetch)
> +            (uri (string-append "https://github.com/hluk/CopyQ/archive/v" version
> +                                ".tar.gz"))

tarballs from github with the '/archive/' string are autogenerated and
can be regenerated in the future, leading to a hash mismatch. I checked
the release page and it looks like our best option is to use the .zip
file. (This also means you'll need to add unzip to native-inputs)

> +            (sha256
> +             (base32
> +              "154yccvlyhr6dml29cvkaqi8277vshmpydvy6ar125qm74q2rib5"))))
> +  (build-system cmake-build-system)
> +  (arguments
> +    `(#:configure-flags '("-DCMAKE_BUILD_TYPE=Release")
> +      #:tests? #f))

Are there no tests or can the test suite not be run? Please leave a note :)

> +  (inputs `(("qtbase" ,qtbase)
> +            ("qtx11extras" ,qtx11extras)
> +            ("qtsvg" ,qtsvg)
> +            ("qtscript" ,qtscript)))
> +  (synopsis "Good qt clipboard manager")

Go ahead and change this to something else. Checking the repo I'd go
with "Clipboard manager with advanced features".

> +  (description "CopyQ is clipboard manager –
> +    a desktop application which stores content of the system clipboard
> +    whenever it changes and allows to search the history
> +    and copy it back to the system clipboard
> +    or paste it directly to other applications.")

Unfortunately the description reads to me like bad poetry :p I copied
some text from the repo and re-wrapped it below:

    (description "CopyQ is clipboard manager with editing and scripting
features.  CopyQ monitors system clipboard and saves its content in customized
tabs.  Saved clipboard can be later copied and pasted directly into any
application.)

> +  (home-page "https://hluk.github.io/CopyQ/")
> +  (license license:gpl3+)))
> +
>  (define-public xclip
>    (package
>      (name "xclip")
> -- 
> 2.24.1
> 


-- 
Efraim Flashner   <efraim@flashner.co.il>   אפרים פלשנר
GPG key = A28B F40C 3E55 1372 662D  14F7 41AA E7DC CA3D 8351
Confidentiality cannot be guaranteed on emails sent or received unencrypted

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

  reply	other threads:[~2019-12-26  6:58 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-12-25 21:07 [bug#38744] [PATCH] adding copyq 3.9.3 to xdisorg.scm Thierry
2019-12-26  6:56 ` Efraim Flashner [this message]
2019-12-28 20:31 ` bug#38744: " Efraim Flashner

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20191226065654.GI23018@E5400 \
    --to=efraim@flashner.co.il \
    --cc=38744@debbugs.gnu.org \
    --cc=thierry@moncourriel.eu \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/guix.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.