From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:470:142:3::10]:43039) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1iiX3e-0007Ik-WB for guix-patches@gnu.org; Sat, 21 Dec 2019 00:12:08 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1iiX3c-0001vf-H1 for guix-patches@gnu.org; Sat, 21 Dec 2019 00:12:06 -0500 Received: from debbugs.gnu.org ([209.51.188.43]:41560) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1iiX3a-0001vH-6X for guix-patches@gnu.org; Sat, 21 Dec 2019 00:12:04 -0500 Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1iiX3a-0003Aj-1y for guix-patches@gnu.org; Sat, 21 Dec 2019 00:12:02 -0500 Subject: [bug#38351] gnome-characters V2 Resent-Message-ID: From: Brett Gilio References: <5e448abe005d25a6c831e20b54cbff6f0610ec63.camel@disroot.org> <3fe2fb92b6c3fcf2b5c5dc7b604395d7f75d8dee.camel@disroot.org> Date: Fri, 20 Dec 2019 23:11:31 -0600 In-Reply-To: <3fe2fb92b6c3fcf2b5c5dc7b604395d7f75d8dee.camel@disroot.org> (Raghav Gururajan's message of "Fri, 20 Dec 2019 22:45:10 -0500") Message-ID: <87zhfmp8jg.fsf@gnu.org> MIME-Version: 1.0 Content-Type: text/plain Content-Transfer-Encoding: quoted-printable List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: guix-patches-bounces+kyle=kyleam.com@gnu.org Sender: "Guix-patches" To: Raghav Gururajan Cc: 38351@debbugs.gnu.org =2D----BEGIN PGP SIGNED MESSAGE----- Hash: SHA256 Raghav Gururajan writes: > Hello Guix! > > Please find the new patch attached with this email, for gnome- > characters. > > Regards, > Raghav Gururajan. > > diff --git a/gnu/packages/gnome.scm b/gnu/packages/gnome.scm > index dab2571fba..9146ffc168 100644 > --- a/gnu/packages/gnome.scm > +++ b/gnu/packages/gnome.scm > @@ -455,6 +455,34 @@ documents. This package also contains binaries that= can convert XPS documents > to other formats.") > (license license:lgpl2.1+))) >=20=20 > +(define-public gnome-characters ;; not gucharmap > + (package > + (name "gnome-characters") ;; not gucharmap > + (version "3.30.0") > + (source (origin > + (method url-fetch) > + (uri (string-append "mirror://gnome/sources/gnome-characters/" > + (version-major+minor version) "/gnome-characters-" version ".tar.x= z")) > + (sha256 > + (base32=20 > + "08cwz39iwgsyyb2wqhb8vfbmh1cwfkgfiy7adp08w7rwqi99x3dp")))) > + (build-system meson-build-system) > + (native-inputs > + `(("glib:bin" ,glib "bin") > + ("gtk+:bin" ,gtk+ "bin") > + ("pkg-config" ,pkg-config))) > + (inputs > + `(("gettext" ,gettext-minimal) > + ("gjs" ,gjs) > + ("gtk+" ,gtk+) > + ("libunistring" ,libunistring))) > + (synopsis "GNOME Characters") > + (description "A simple utility application to find and=20 > + insert unusual characters. It allows you to quickly find=20 > + the character you are looking for by searching for keywords.") > + (home-page "https://wiki.gnome.org/Design/Apps/CharacterMap") > + (license license:bsd-3)) > + > (define-public gnome-common > (package > (name "gnome-common") > Hi Raghav, First, thank you for your submission. I have a few questions and a few suggestions. I am curious about why you added this comment? =2D --8<---------------cut here---------------start------------->8--- (define-public gnome-characters ;; not gucharmap =2D --8<---------------cut here---------------end--------------->8--- While I understand where you are coming from, this comment doesn't really add much information to the code itself. This seems like something maybe to disclaim in the package description, if that is even necessary. Just a thought. Second, your description features an indentation that is not needed. The `guix lint` command should catch this with the ./pre-inst-env for your package. If you need help with this, or other stylization for how to format synopsis and descriptions, feel free to ask. Lastly, on my end this doesn't appear to be in patch format, but rather seems like a diff with a .patch extension. We generally use `git format-patch` for generating patches, which you can find in our documentation accompanied by a properly formatted commit message and sub-header. If you would like some links to the documentation on how to obtain this, and maybe some tips and tricks on achieving this from GNU Emacs (though not required), please do let me know. I am happy to help you, and would like to see your changes be reflected upstream. Thank you a lot for your work. =2D --=20 Brett M. Gilio GNU Guix, Contributor | GNU Project, Webmaster [DFC0 C7F7 9EE6 0CA7 AE55 5E19 6722 43C4 A03F 0EEE] =2D----BEGIN PGP SIGNATURE----- iQIzBAEBCAAdFiEE38DH957mDKeuVV4ZZyJDxKA/Du4FAl39qYMACgkQZyJDxKA/ Du7l2g//UZfo5es5HfddQDWOZc0hGYt3Ih3QvBTZVu3o0/r2tqokEnzvYLwmhkmS ZuZXS1MRdomGJZ5l80w6EmwESYWIosWewAzDPbCHHBMg+Dyh4omd8Lj7Bh6JF1aK H14LoQfsc8U86vGlh3FyjXnOq589KZLiNplewnTw/aC1IdQwjsSkIBZOm5b1V+A6 KUPgbgQJja6rqcsZNcXxUxbx/iTuSJxDsp46jGUCOK9TU2AI2ekiI5awP5TyXkQw LSicDFfF+hpNkXBXPILDa24B3273qE4f7Gm/rngignrVsxZTPwJOLw3CAaYkeMqh xkxAfh681l8ztdZ9Cmez3RA0CJBEBjIOy+TtB2iJWDkjdH294T1DETlxb5t2H7Qj pT8ldWRdGtd10MiH0mJbQ3qwSdFPKL+XwwOigKViuUo34eMIZgmXSOVXO0OtC0Ml fPsOEWnlISXVKdNPIW8pdqoXcRyB4DSl1ZTLw4/BjqI/qXSdUTV077/SC7/HlGn/ Vekpe1sM8koVnAwxbItKKy4+geGuX4Dn1uhgwzbGpnXTNl4xIl1NWJTwuA3SN5Zp ccx3gqLDShH3a2k7RLM30HUPm53Y8U3M+TXw+SztVbkpgnrQmjlDlySDjc9/vbjP j9Yb3nno/TW5sbT7r/pivj+93aX+BjReSMxJs5a9pTvGLA9uFnc=3D =3Dh6gx =2D----END PGP SIGNATURE-----