1235395 - whine.pl broken due to a missing generate_email() routine

Status: RESOLVED FIXED

(Bugzilla :: Whining, defect, P1)

Description

[Marcus von Appen]

User Agent: Mozilla/5.0 (X11; FreeBSD amd64; rv:43.0) Gecko/20100101 Firefox/43.0
Build ID: 20151225125557

Steps to reproduce:

Execute the whine.pl cronjob according to http://bugzilla.readthedocs.org/en/5.0/installing/optional-post-install-config.html?highlight=whine


Actual results:

the script broke due to "Undefined subroutine &main::generate_email called at ./whine.pl line 355"


Expected results:

Working whine mails should have been sent

It looks like there is no generate_email() routine defined in Bugzilla/Mailer.pm in 5.0.2., while there is one in the current head/master

Comment 1

[Frédéric Buclin]

Regression in 5.0.2 when committing bug 714724. 5.0.1 is fine.

Comment 2

[Dylan Hardison [:dylan] (he/him)]

Attachment: 1235395_2.patch

Comment 3

[Dylan Hardison [:dylan] (he/him)]

Attachment: 1235395_4.patch

Comment 4

[Dylan Hardison [:dylan] (he/him)]

Attachment: 1235395_5.patch

Fixed recursive import error

Comment 5

[Frédéric Buclin]

Comment on attachment 8702436 [details] [diff] [review]
1235395_5.patch

Looks good and works fine. The comments below are all nits, but would be good to be fixed on commit to avoid future conflicts when backporting fixes, and to keep consistency within our codebase.


>+++ b/Bugzilla/Mailer.pm

>-@Bugzilla::Mailer::EXPORT = qw(MessageToMTA build_thread_marker);
>-
>+@Bugzilla::Mailer::EXPORT = qw(MessageToMTA build_thread_marker generate_email);

No reason to remove the empty line. Everywhere else, we leave an empty line after ::EXPORT, for readability. Would be good to do it here too, also for consistency.


> use Bugzilla::Util;
>+use Bugzilla::User;

In master, Bugzilla::User is put just above Bugzilla::Util. Doing it here too would avoid useless conflicts in the future when trying to apply a patch from master to the 5.0 branch. It's always painful to have to backport a patch for such minor reasons.


>+sub generate_email {

Here too, generate_email() is put as the first method in master, before MessageToMTA(). Here you put it last. :(



>+++ b/whine.pl
> use warnings;
>-
> use lib qw(. lib);

No reason to remove the empty line. Here too, we always leave an empty line between both, for readability and consistency.


>-use Bugzilla::Mailer;
>+use Bugzilla::Mailer qw(MessageToMTA generate_email);

Here too, this change is useless, and isn't done upstream.


r=LpSolit, but please keep consistency with master (to ease backports) and the remaining code (for consistency).

Comment 6

[Dylan Hardison [:dylan] (he/him)]

To ssh://[e-mail]/bugzilla/bugzilla.git
   15cc087..26a693a  5.0 -> 5.0

Comment 7

[abhishek]

Hi,

Follow all steps but still emails are not going through whining.

Can any one help me out

Bugzilla version :- 5.0.2

Comment 8

[Andre Klapper]

@abhishek: As your issue is likely not related to this specific bug report, please see http://www.bugzilla.org/support/ - Thanks!

Comment 9

[abhishek]

Dear Ander,

Issue :- 
After upgrading bugzilla from 4.4.4 to 5.0.2 version whining script not running.

Error :- undefined subroutine &main: generate_email called at ./whine.pl line 355

kindly suggest how to resolve the error.

Thanks in advance

Comment 10

[abhishek]

Dear Ander,

Can you suggest how to resolve the issue ?

I am waiting for your valuable feedback.

Thanks

Comment 11

[Andre Klapper]

Please see comment 8 again for venues. Your problem is off-topic for this bug report. Thanks.

Comment 12

[Frédéric Buclin]

His question is related to this issue, but questions about how to apply a patch should indeed not be asked here but rather in the support mailing-list (or simply read the documentation).

Either you apply the patch manually (patch -p1 -i name_of_the_patch.diff) or you wait for Bugzilla 5.0.3 which will have this fix.

Comment 13

[abhishek]

Issue Resolved,

Thanks a ton Guys.