From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:40641) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1dXCs9-0002HW-Gf for guix-patches@gnu.org; Mon, 17 Jul 2017 16:44:06 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1dXCs6-0001Yp-Oh for guix-patches@gnu.org; Mon, 17 Jul 2017 16:44:05 -0400 Received: from debbugs.gnu.org ([208.118.235.43]:42220) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1dXCs6-0001YY-KN for guix-patches@gnu.org; Mon, 17 Jul 2017 16:44:02 -0400 Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1dXCs6-0006zS-7A for guix-patches@gnu.org; Mon, 17 Jul 2017 16:44:02 -0400 Subject: [bug#25728] [PATCH 2/2] gnu: Add colorforth. Resent-Message-ID: References: <20170214185339.25538-1-contact.ng0@cryptolab.net> <20170214185339.25538-2-contact.ng0@cryptolab.net> <871suvu8yu.fsf@elephly.net> <20170415152843.2vyk7uhycuny5rov@abyayala> <20170415160053.qqqpga27hqsdhet3@abyayala> <87d1a17e94.fsf@elephly.net> <20170618115947.jo22jym5bm6semng@abyayala> <87shix4d40.fsf@elephly.net> <20170628094810.nt4svpdksevrh5gr@abyayala> <877ez6hkbk.fsf@elephly.net> From: Ricardo Wurmus In-reply-to: <877ez6hkbk.fsf@elephly.net> Date: Mon, 17 Jul 2017 22:43:06 +0200 Message-ID: <8760eqhk11.fsf@elephly.net> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit 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: ng0 Cc: 25728@debbugs.gnu.org Ricardo Wurmus writes: > It won’t work when it’s in the store because we cannot write to the > store. Even after copying it out of the store and changing permissions > it would simply crash. This could be a problem with my Libreboot > machine, but the first problem is bad enough. > > If the software cannot be used when it is installed to the store then I > don’t think it should be in the store at all. I think we should drop > this patch as the package is not actually usable. > > What do you think? The alternative is to also install a wrapper script to “bin” that essentially does this: #!/bin/sh qemu-system-i386 -snapshot -boot a -fda $prefix/share/colorforth/cf2012.img It still crashes for me but at least it doesn’t complain about store permissions. If you can get this to run on your computer then please send a final patch that incorporates all of these suggestions. Otherwise I think it may not be worth spending more time on a toy package like this. What do you think? -- Ricardo GPG: BCA6 89B6 3655 3801 C3C6 2150 197A 5888 235F ACAC https://elephly.net