From mboxrd@z Thu Jan 1 00:00:00 1970 From: ludo@gnu.org (Ludovic =?UTF-8?Q?Court=C3=A8s?=) Subject: bug#31560: Commit 47a60325c broke tests/pack.scm Date: Fri, 01 Jun 2018 15:41:46 +0200 Message-ID: <874limaajp.fsf@gnu.org> References: <87bmd6zyvd.fsf@gmail.com> <87lgca3jd0.fsf@gnu.org> <87zi0f9cgx.fsf@gmail.com> 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]:58203) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1fOkKh-0006DK-Q0 for bug-guix@gnu.org; Fri, 01 Jun 2018 09:43:08 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1fOkKc-0001Po-LY for bug-guix@gnu.org; Fri, 01 Jun 2018 09:43:07 -0400 Received: from debbugs.gnu.org ([208.118.235.43]:49077) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1fOkKc-0001PX-IF for bug-guix@gnu.org; Fri, 01 Jun 2018 09:43:02 -0400 Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1fOkKc-0000II-Bw for bug-guix@gnu.org; Fri, 01 Jun 2018 09:43:02 -0400 Sender: "Debbugs-submit" Resent-To: bug-guix@gnu.org Resent-Message-ID: In-Reply-To: <87zi0f9cgx.fsf@gmail.com> (Chris Marusich's message of "Fri, 01 Jun 2018 00:45:34 -0700") List-Id: Bug reports for GNU Guix List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: bug-guix-bounces+gcggb-bug-guix=m.gmane.org@gnu.org Sender: "bug-Guix" To: Chris Marusich Cc: 31560-done@debbugs.gnu.org Hello, Chris Marusich skribis: > I believe the attached patches fix the issue when applied to the master > branch. The failing test now passes (along with all the other "make > check" tests). What do you think? If you approve, then I'll commit it > to the master branch. I looked some more and the patch would create broken links. I concluded that the issue was simply that the test needed to be adjusted to expect a relative symlink instead of an absolute symlink. I first cherry-picked the fix that Ricardo made in =E2=80=98core-updates=E2= =80=99 (same as the one you posted) as commit 44057a461b1fa8102938c4e0f54d7cbc9dd09b03. Then I adjusted the test in commit ccc951cab3172adfdaf6fd2dfa8f8cdb98358a69. Let me know if you think something is amiss! Thank you, Ludo=E2=80=99.