From mboxrd@z Thu Jan 1 00:00:00 1970 From: ludo@gnu.org (Ludovic =?utf-8?Q?Court=C3=A8s?=) Subject: Re: [PATCH] gnu: Add higan. Date: Fri, 10 Jun 2016 14:56:07 +0200 Message-ID: <87h9d119ug.fsf@gnu.org> References: <87fustalcn.fsf@T420.taylan> <87porrkgab.fsf@gnu.org> <87fusn7r5i.fsf@T420.taylan> <87k2hzem54.fsf@gnu.org> <8737om6qkd.fsf@T420.taylan> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Return-path: Received: from eggs.gnu.org ([2001:4830:134:3::10]:48249) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bBLyw-0002sG-W4 for guix-devel@gnu.org; Fri, 10 Jun 2016 08:56:16 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1bBLys-0006jq-U4 for guix-devel@gnu.org; Fri, 10 Jun 2016 08:56:15 -0400 In-Reply-To: <8737om6qkd.fsf@T420.taylan> ("Taylan Ulrich \=\?utf-8\?Q\?\=5C\=22Bay\=C4\=B1rl\=C4\=B1\=2FKammer\=5C\=22\=22's\?\= message of "Thu, 09 Jun 2016 23:43:46 +0300") List-Id: "Development of GNU Guix and the GNU System distribution." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: guix-devel-bounces+gcggd-guix-devel=m.gmane.org@gnu.org Sender: "Guix-devel" To: Taylan Ulrich =?utf-8?Q?=22Bay=C4=B1rl=C4=B1?= =?utf-8?Q?=2FKammer=22?= Cc: guix-devel@gnu.org taylanbayirli@gmail.com (Taylan Ulrich "Bay=C4=B1rl=C4=B1/Kammer") skribis: > I contacted the maintainer of the repo and asked them to add a tag, > which is done now. Great! > Note: apparently the author re-released 098 with a hotfix very soon > after releasing it, without making a version change. In the repository, > this is tagged as 098b, which I use here. I.e. this is not a true > version update; we were already using "this 098". Not great. ;-) >>> Apparently, the files that are part of the distribution are pure data >>> files, i.e. fine to be read-only. However, the directory hierarchy of >>> which they're a part needs to be writable, as higan creates further >>> files there. With that cp -r, the directory hierarchy is made sure to >>> be there, and the data files made sure to be up to date. >>> >>> Although I didn't look too closely at the sources, patching higan to do >>> things differently would presumably be a nontrivial task, since it seems >>> bent on doing things in terms of this directory structure that contains >>> both pure data and read-write data files. >> >> Hmm OK. What do other distros do? > > More or less the same thing; it's sanctioned by the developers even > though it's acknowledged not to be a particularly good solution. Here > we see Arch do the same: > > https://git.archlinux.org/svntogit/community.git/tree/trunk/higan?h=3Dpac= kages/higan > > (Though this one seems to have a bug because they copy things to > ~/Emulation instead of the new ~/.local/share. I'm confident that > ~/.local/share is the right location in the latest release. Also for > the cheats.bml file.) Hmmk. > Here's a patch that, in addition to switching the repo, adds a patch > removing a -march=3Dnative flag (which I assume would break > cross-compilation), =E2=80=98-march=3Dnative=E2=80=99 breaks distribution of pre-built binaries= since the binary produced in the build farm may or may not run on the user=E2=80=99s hardware, depending on the exact CPU being used. So this patch is the right thing to do! > and passes the make flag "profile=3Dbalanced" to make higan less CPU > intensive by emulating a bit less accurately (it was running too slow > on my i5, so this should be a better default). OK. > I just saved a princess from an evil wizard with this so I can confirm > it runs smooth now. Wonderful. :-) > From 93ccdf075caf462af742cf46cad35697001db124 Mon Sep 17 00:00:00 2001 > From: =3D?UTF-8?q?Taylan=3D20Ulrich=3D20Bay=3DC4=3DB1rl=3DC4=3DB1/Kammer?= =3D > > Date: Thu, 9 Jun 2016 21:53:02 +0300 > Subject: [PATCH] gnu: higan: Various improvements. > > * gnu/packages/games.scm (higan): Use semi-official repository at GitLab > (using hotfix tag 098b which is equivalent to official release 098). > Add a patch to remove the build flag -march=3Dnative. Set profile to > balanced. > * gnu/packages/patches/higan-remove-march-native-flag.patch: New file. > * gnu/local.mk (dist_patch_DATA): Add it. OK! Thanks, Ludo=E2=80=99.