GNU bug report logs - #51023
[PATCH] gnu: php: patch failing test case

Previous Next

Package: guix-patches;

Reported by: Ryan Sundberg <ryan <at> arctype.co>

Date: Tue, 5 Oct 2021 04:20:02 UTC

Severity: normal

Tags: patch

Done: Tobias Geerinckx-Rice <me <at> tobias.gr>

Bug is archived. No further changes may be made.

Full log


Message #10 received at 51023-done <at> debbugs.gnu.org (full text, mbox):

From: Tobias Geerinckx-Rice <me <at> tobias.gr>
To: Ryan Sundberg <ryan <at> arctype.co>
Cc: 51023-done <at> debbugs.gnu.org
Subject: Re: [bug#51023] [PATCH] gnu: php: patch failing test case
Date: Tue, 05 Oct 2021 13:47:24 +0200
[Message part 1 (text/plain, inline)]
Ryan,

Ryan Sundberg via Guix-patches via 写道:
> Fixes a failing test case in php (Zend/tests/bug74093.phpt)
> See https://github.com/php/php-src/pull/7555/files for upstream 
> status.

Thanks!  PHP is notorious for

 sleep 4 # this is how long x took to y on my laptop in 2014

but it's the bane of many test suites (and, hence, us)…

> * gnu/local.mk: Include patch file php-bug-74093-test.patch
> * gnu/packages/patches/php-bug-74093-test.patch: Patch file
> * gnu/packages/php.scm: Add patch to php

Thanks for writing a GNU/Guix-style commit message.  I made only a 
few tweaks, mainly adding full stops and (section)[things].

   gnu: php: Patch failing test case.

   Fixes a failing test case in PHP (Zend/tests/bug74093.phpt). 
   See
   <https://github.com/php/php-src/pull/7555/files> for upstream 
   status.

   * gnu/packages/php.scm (php)[source]: Add patch.
   * gnu/packages/patches/php-bug-74093-test.patch: New file.
   * gnu/local.mk (dist_patch_DATA): Add it.

The * line order is just what I'm used to writing, nothing more.

Pushed as 9aaf402a379b9ac9e4c3eee5b16f7496d7606e91!

Kind regards,

T G-R
[signature.asc (application/pgp-signature, inline)]

This bug report was last modified 3 years and 225 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.