travis ci failing: Error: Can't locate Email/Send.pm in @INC

RESOLVED FIXED

Status

()

Bugzilla
Testing Suite
--
major
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: glob, Assigned: dkl)

Tracking

({regression})

regression

Details

(URL)

(Reporter)

Description

3 years ago
#   Failed test 'use Bugzilla;'
#   at t/001compile.t line 23.
#     Tried to use 'Bugzilla'.
#     Error:  Can't locate Email/Send.pm in @INC (@INC contains: . lib t /home/travis/perl5/perlbrew/perls/5.10/lib/5.10.1/x86_64-linux /home/travis/perl5/perlbrew/perls/5.10/lib/5.10.1 /home/travis/perl5/perlbrew/perls/5.10/lib/site_perl/5.10.1/x86_64-linux /home/travis/perl5/perlbrew/perls/5.10/lib/site_perl/5.10.1) at Bugzilla/Mailer.pm line 32.

the change which triggered this didn't touch Email::Send
https://github.com/bugzilla/bugzilla/compare/981729e5b625...3fbd84bb4808
making this a travis-ci issue.

Comment 1

3 years ago
(In reply to Byron Jones ‹:glob› from comment #0)
> the change which triggered this didn't touch Email::Send
> making this a travis-ci issue.

If this is a Travis CI issue, why reporting it here?
(Reporter)

Comment 2

3 years ago
it's an issue with our travis-ci stuff, as opposed to the changes i made.
(Assignee)

Comment 3

3 years ago
This is related to a change I made recently to significantly speed up the Travis tests. I missed the Email::Send module for 4.4 branches and older. This is fixed now in the QA repos and should be good on the next builds. 

Currently looking at a new problem with PostgreSQL 9.4 on Ubuntu 14.04 causing the Pg tests to fail but that is a different issue.

dkl
Assignee: testing → dkl
Status: NEW → RESOLVED
Last Resolved: 3 years ago
Keywords: regression
OS: Mac OS X → All
Hardware: x86 → All
Resolution: --- → FIXED

Comment 4

3 years ago
Why isn't 5.0 updated as well? travis.sh is the same in master and 4.4, but 5.0 has completely different code. Also, 5.0 and master do not depend on Email::Send anymore; they use Email::Sender instead. And 4.4 depends on Email::Send, not Email::Sender.
(Assignee)

Comment 5

3 years ago
(In reply to Frédéric Buclin from comment #4)
> Why isn't 5.0 updated as well? travis.sh is the same in master and 4.4, but
> 5.0 has completely different code. Also, 5.0 and master do not depend on
> Email::Send anymore; they use Email::Sender instead. And 4.4 depends on
> Email::Send, not Email::Sender.

As for 5.0, that was an oversight that is fixed now. I synced the travis script changes to all branches except 5.0 accidentally. As I use the same travis.sh script to set up the Travis testing VMs, I also install Email::Sender (and now Email::Send) in all VMs regardless of branch. It is safe to have both installed at the same time and the code will use the one it needs.

dkl
You need to log in before you can comment on or make changes to this bug.