From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([209.51.188.92]:48274) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1gtuNW-0001nz-CY for guix-patches@gnu.org; Wed, 13 Feb 2019 08:15:07 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1gtuNS-0004EK-1M for guix-patches@gnu.org; Wed, 13 Feb 2019 08:15:06 -0500 Received: from debbugs.gnu.org ([209.51.188.43]:46606) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1gtuNR-0004EA-Tc for guix-patches@gnu.org; Wed, 13 Feb 2019 08:15:01 -0500 Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1gtuNR-0005Nr-O7 for guix-patches@gnu.org; Wed, 13 Feb 2019 08:15:01 -0500 Subject: [bug#34378] update GNOME Planner to current Resent-Message-ID: References: <9bb349a5-a9a9-13b2-35c4-1ea63ffcbd43@interpretmath.pw> <87h8d8hdnw.fsf@gnu.org> From: Andrew Miloradovsky Message-ID: <90daaef6-902f-f6a5-7cdb-9e3b162110c1@interpretmath.pw> Date: Wed, 13 Feb 2019 13:13:59 +0000 MIME-Version: 1.0 In-Reply-To: <87h8d8hdnw.fsf@gnu.org> Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit Content-Language: en-US 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: Ludovic =?UTF-8?Q?Court=C3=A8s?= Cc: 34378@debbugs.gnu.org Hi Ludovic, On 2/12/19 9:58 PM, Ludovic Courtès wrote: > Hi Andrew, > > Andrew Miloradovsky skribis: > >> The release is fairly old and contains prebuilt scripts. > What do you mean by “prebuilt scripts”? By pre-built scripts I mean things like `configure`, generated by auto-tools and gnome-common. >> From cbe5c7a354381c665735085a5eb262404e901385 Mon Sep 17 00:00:00 2001 >> From: Andrew Miloradovsky >> Date: Fri, 8 Feb 2019 00:03:48 +0000 >> Subject: [PATCH] gnu: gnome-planner: update 0.14.6 to 2019-02-08 >> >> Build all from a modern version, enable Python bindings. >> >> * gnu/packages/gnome.scm (gnome-planner): >> - Fetch the sources from GNOME's own GitLab. >> - Run `autogen.sh` with the proper shell paths. >> - Enable Python 2 bindings, because we can now. >> - Add all the inputs necessary for all of that. > Please see ‘git log’ and > > for examples of the commit message format. The changes made didn't fit well in the recommended commit message structure (documenting the changes to each field separately). >> + ;;; Warning: >> + ;;; The documentation generation mechanism is long broken. No Yelp yet. >> + ;;; So the output doesn't contain any docs, unlike the ancient release. >> + ;;; OTOH, that's probably not a huge concern, given the circumstances. >> + (let ((commit "fa7cbe309d5a705502ca46f808bcf78840804dbe") >> + (revision "2019-02-08")) > How did you pick this particular commit? In general we only package > releases, unless there’s a compelling reason to build straight from the > VCS. Do you think that’s the case here? I picked that commit just because it was the most recent: the release is expected to be very difficult to make, a lot of things need to be fixed/modernized first, and there are not many people to review the patches (one to be exact, AFAICT). I'm not sure there are really compelling reasons: most/all of the changes made since the release are either about translations (documentation generation is broken anyways), or fixes of deprecation warnings (which otherwise would prevent it from building in Nixpkgs with the default -Werror). > > Thanks for the patch! > > Ludo’.