git@vger.kernel.org mailing list mirror (one of many)
 help / color / mirror / code / Atom feed
* t9001 failures on 32-bit Linux?
@ 2017-12-10 19:58 Ramsay Jones
  2017-12-10 20:33 ` Ævar Arnfjörð Bjarmason
  2017-12-10 22:28 ` Junio C Hamano
  0 siblings, 2 replies; 5+ messages in thread
From: Ramsay Jones @ 2017-12-10 19:58 UTC (permalink / raw)
  To: Junio C Hamano; +Cc: GIT Mailing-list

Hi Junio,

I noticed the revert of the 'ab/simplify-perl-makefile' branch on
top of 'pu'. So, I fired up my 32-bit Linux and attempted to see
if I could debug this t9001 test failure.

Unfortunately, I could not get it to fail. :(

Both of the 'pu' (@77e921d77d) and 'pu~1' (@cfef1ebefd) builds pass
the full test-suite and, likewise, running t9001 in a loop for a
count of 100 (about 45 minutes run-time each).

[If it makes any difference, I don't have sendmail installed (no
/usr/sbin/sendmail or /usr/lib/sendmail), or any sendmail in my
$PATH.]

Sorry I couldn't help with this! :(

ATB,
Ramsay Jones




^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: t9001 failures on 32-bit Linux?
  2017-12-10 19:58 t9001 failures on 32-bit Linux? Ramsay Jones
@ 2017-12-10 20:33 ` Ævar Arnfjörð Bjarmason
  2017-12-10 21:02   ` Ramsay Jones
  2017-12-10 22:28 ` Junio C Hamano
  1 sibling, 1 reply; 5+ messages in thread
From: Ævar Arnfjörð Bjarmason @ 2017-12-10 20:33 UTC (permalink / raw)
  To: Ramsay Jones; +Cc: Junio C Hamano, GIT Mailing-list

On Sun, Dec 10, 2017 at 8:58 PM, Ramsay Jones
<ramsay@ramsayjones.plus.com> wrote:
> I noticed the revert of the 'ab/simplify-perl-makefile' branch on
> top of 'pu'. So, I fired up my 32-bit Linux and attempted to see
> if I could debug this t9001 test failure.
>
> Unfortunately, I could not get it to fail. :(
>
> Both of the 'pu' (@77e921d77d) and 'pu~1' (@cfef1ebefd) builds pass
> the full test-suite and, likewise, running t9001 in a loop for a
> count of 100 (about 45 minutes run-time each).
>
> [If it makes any difference, I don't have sendmail installed (no
> /usr/sbin/sendmail or /usr/lib/sendmail), or any sendmail in my
> $PATH.]
>
> Sorry I couldn't help with this! :(

It's a bug in my patch, I'll follow-up with a patch once I figure out
what it is, but for now I wanted to say it's my bad.

That this is failing has nothing to do with 32bit per say, but that
system doesn't have Error.pm installed, so we fall back on our own
copy, there's some bug in the new Error.pm fallback logic I
introduced.

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: t9001 failures on 32-bit Linux?
  2017-12-10 20:33 ` Ævar Arnfjörð Bjarmason
@ 2017-12-10 21:02   ` Ramsay Jones
  2017-12-10 21:14     ` Ævar Arnfjörð Bjarmason
  0 siblings, 1 reply; 5+ messages in thread
From: Ramsay Jones @ 2017-12-10 21:02 UTC (permalink / raw)
  To: Ævar Arnfjörð Bjarmason; +Cc: Junio C Hamano, GIT Mailing-list



On 10/12/17 20:33, Ævar Arnfjörð Bjarmason wrote:
> On Sun, Dec 10, 2017 at 8:58 PM, Ramsay Jones
> <ramsay@ramsayjones.plus.com> wrote:
>> I noticed the revert of the 'ab/simplify-perl-makefile' branch on
>> top of 'pu'. So, I fired up my 32-bit Linux and attempted to see
>> if I could debug this t9001 test failure.
>>
>> Unfortunately, I could not get it to fail. :(
>>
>> Both of the 'pu' (@77e921d77d) and 'pu~1' (@cfef1ebefd) builds pass
>> the full test-suite and, likewise, running t9001 in a loop for a
>> count of 100 (about 45 minutes run-time each).
>>
>> [If it makes any difference, I don't have sendmail installed (no
>> /usr/sbin/sendmail or /usr/lib/sendmail), or any sendmail in my
>> $PATH.]
>>
>> Sorry I couldn't help with this! :(
> 
> It's a bug in my patch, I'll follow-up with a patch once I figure out
> what it is, but for now I wanted to say it's my bad.
> 
> That this is failing has nothing to do with 32bit per say, but that
> system doesn't have Error.pm installed, so we fall back on our own
> copy, there's some bug in the new Error.pm fallback logic I
> introduced.

Ah, OK, that makes sense.

Thanks!

ATB,
Ramsay Jones



^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: t9001 failures on 32-bit Linux?
  2017-12-10 21:02   ` Ramsay Jones
@ 2017-12-10 21:14     ` Ævar Arnfjörð Bjarmason
  0 siblings, 0 replies; 5+ messages in thread
From: Ævar Arnfjörð Bjarmason @ 2017-12-10 21:14 UTC (permalink / raw)
  To: Ramsay Jones; +Cc: Junio C Hamano, GIT Mailing-list


On Sun, Dec 10 2017, Ramsay Jones jotted:

> On 10/12/17 20:33, Ævar Arnfjörð Bjarmason wrote:
>> On Sun, Dec 10, 2017 at 8:58 PM, Ramsay Jones
>> <ramsay@ramsayjones.plus.com> wrote:
>>> I noticed the revert of the 'ab/simplify-perl-makefile' branch on
>>> top of 'pu'. So, I fired up my 32-bit Linux and attempted to see
>>> if I could debug this t9001 test failure.
>>>
>>> Unfortunately, I could not get it to fail. :(
>>>
>>> Both of the 'pu' (@77e921d77d) and 'pu~1' (@cfef1ebefd) builds pass
>>> the full test-suite and, likewise, running t9001 in a loop for a
>>> count of 100 (about 45 minutes run-time each).
>>>
>>> [If it makes any difference, I don't have sendmail installed (no
>>> /usr/sbin/sendmail or /usr/lib/sendmail), or any sendmail in my
>>> $PATH.]
>>>
>>> Sorry I couldn't help with this! :(
>>
>> It's a bug in my patch, I'll follow-up with a patch once I figure out
>> what it is, but for now I wanted to say it's my bad.
>>
>> That this is failing has nothing to do with 32bit per say, but that
>> system doesn't have Error.pm installed, so we fall back on our own
>> copy, there's some bug in the new Error.pm fallback logic I
>> introduced.
>
> Ah, OK, that makes sense.

Updated patch sent just now as <20171210211333.9820-1-avarab@gmail.com>:
https://public-inbox.org/git/20171210211333.9820-1-avarab@gmail.com/

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: t9001 failures on 32-bit Linux?
  2017-12-10 19:58 t9001 failures on 32-bit Linux? Ramsay Jones
  2017-12-10 20:33 ` Ævar Arnfjörð Bjarmason
@ 2017-12-10 22:28 ` Junio C Hamano
  1 sibling, 0 replies; 5+ messages in thread
From: Junio C Hamano @ 2017-12-10 22:28 UTC (permalink / raw)
  To: Ramsay Jones; +Cc: GIT Mailing-list

Ramsay Jones <ramsay@ramsayjones.plus.com> writes:

> Hi Junio,
>
> I noticed the revert of the 'ab/simplify-perl-makefile' branch on
> top of 'pu'. So, I fired up my 32-bit Linux and attempted to see
> if I could debug this t9001 test failure.
>
> Unfortunately, I could not get it to fail. :(
>
> Both of the 'pu' (@77e921d77d) and 'pu~1' (@cfef1ebefd) builds pass
> the full test-suite and, likewise, running t9001 in a loop for a
> count of 100 (about 45 minutes run-time each).
>
> [If it makes any difference, I don't have sendmail installed (no
> /usr/sbin/sendmail or /usr/lib/sendmail), or any sendmail in my
> $PATH.]
>
> Sorry I couldn't help with this! :(

I suspect that the difference is not about send(e)mail per-se, but
the differences between the set of perl modules installed in the
linux32 Travis environment and the other one.  Perhaps the use of
deprecated Error.pm is the culprit?

I do appreciate that the environments we use on Travis are not
monoculture and we end up testing more variations (in this case,
the cause likely does not have anything to do with 64- vs 32-bit,
but Perl modules).  It sometimes gets confusing, though ;-)

Another difference I noticed while scanning the failing log is that
the Linux32 environment seem to lack libsvn-perl and all tests are
skipped there.  I do not think that this particular variation is
helping us to diversify our test base, though ;-)

Thanks.

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2017-12-10 22:28 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-12-10 19:58 t9001 failures on 32-bit Linux? Ramsay Jones
2017-12-10 20:33 ` Ævar Arnfjörð Bjarmason
2017-12-10 21:02   ` Ramsay Jones
2017-12-10 21:14     ` Ævar Arnfjörð Bjarmason
2017-12-10 22:28 ` Junio C Hamano

Code repositories for project(s) associated with this public inbox

	https://80x24.org/mirrors/git.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).