Bug 2923 - build failure with perl 5.18
Summary: build failure with perl 5.18
Status: RESOLVED FIXED
Alias: None
Product: Claws Mail (GTK 2)
Classification: Unclassified
Component: Plugins/Perl (show other bugs)
Version: 3.9.1
Hardware: All Linux
: P3 normal
Assignee: users
URL: http://bugs.debian.org/708004
Depends on:
Blocks:
 
Reported: 2013-05-13 08:17 UTC by Ricardo Mones
Modified: 2015-07-16 02:21 UTC (History)
0 users

See Also:


Attachments

Description Ricardo Mones 2013-05-13 08:17:13 UTC
Seems the perl POD parser has become more strict and the perl plugin fails to build with new perl 5.18 from Debian experimental (will be uploaded to unstable soon).

Quoting original report:
-----
src/plugins/perl/cm_perl.pod around line 591: Expected text after =item, not a number                                                                           
src/plugins/perl/cm_perl.pod around line 596: Expected text after =item, not a number                                                                           
src/plugins/perl/cm_perl.pod around line 600: Expected text after =item, not a number                                                                           
POD document had syntax errors at /usr/bin/pod2man line 71.
-----

thanks in advance,
Comment 1 users 2013-05-13 22:06:03 UTC
Changes related to this bug have been committed.
Please check latest Git and update the bug accordingly.
You can also get the patch from:
http://git.claws-mail.org/

http://git.claws-mail.org/?p=claws.git;a=commitdiff;h=e77b9943b358fa19d29061358191f1c9e463e998
Author: H.Merijn Brand <h.m.brand@xs4all.nl>
Date:   Mon May 13 11:16:08 2013 +0200

    Fix bug #2923 'build failure with perl 5.18'
    
    Removed also some unnecessary whitespaces
Comment 2 Ricardo Mones 2013-08-13 06:33:05 UTC
Reopening because original bug was reopened as it still fails to build with newer perl.

Original reporter provided a new patch fixing the remaining instances of the error that causes the failure:

http://bugs.debian.org/cgi-bin/bugreport.cgi?msg=38;filename=claws-pod.patch;att=1;bug=708004
Comment 3 users 2013-08-14 15:23:04 UTC
Changes related to this bug have been committed.
Please check latest Git and update the bug accordingly.
You can also get the patch from:
http://git.claws-mail.org/

http://git.claws-mail.org/?p=claws.git;a=commitdiff;h=fa243ae92017fc99f5be788debdc40aadf0a19b6
Author: Ricardo Mones <ricardo@mones.org>
Date:   Wed Aug 14 17:19:55 2013 +0200

    Fix remaining cases of bug #2923
    
    Patch thanks to Dominic Hargreaves <dom@earth.li>
    on http://bugs.debian.org/708004
Comment 4 users 2015-07-16 02:20:04 UTC
Changes related to this bug have been committed.
Please check latest Git and update the bug accordingly.
You can also get the patch from:
http://git.claws-mail.org/

++ ChangeLog	2015-07-16 04:20:02.787125573 +0200
http://git.claws-mail.org/?p=claws.git;a=commitdiff;h=146582502539739159668698d34a0577cccee8b2
Merge: 9b3e3b8 7c2dd3d
Author: Colin Leroy <colin@colino.net>
Date:   Thu Jul 16 04:20:02 2015 +0200

    Merge branch 'master' of file:///home/git/claws

http://git.claws-mail.org/?p=claws.git;a=commitdiff;h=7c2dd3d976388c353632ea0c3199ccdd3e214530
Author: Charles Lehner <cel@celehner.com>
Date:   Wed Jul 15 22:14:59 2015 -0400

    ManageSieve: fix manager window button spacing
    
    - Remove unused vbox
    - Move refresh button to bottom, since it is semantically different
      from the other action buttons

http://git.claws-mail.org/?p=claws.git;a=commitdiff;h=2328d40cf23a5a6c50417bc266e42c5b2b3a90bc
Author: Charles Lehner <cel@celehner.com>
Date:   Wed Jul 15 21:57:32 2015 -0400

    ManageSieve: remove Activate button
    
    A filter can be activated by selecting its radio button in the list,
    so the activate button is not needed.
    
    Bug #3469 progress

http://git.claws-mail.org/?p=claws.git;a=commitdiff;h=455254fe241f14694c85d56c6c7c3e931b8da30f
Author: Charles Lehner <cel@celehner.com>
Date:   Wed Jul 15 21:47:45 2015 -0400

    ManageSieve: save manager window size (bug #2923)

http://git.claws-mail.org/?p=claws.git;a=commitdiff;h=ae7bea75db69009e0dae20c4262e03f9ea74c8d9
Author: Charles Lehner <cel@celehner.com>
Date:   Wed Jul 15 20:31:51 2015 -0400

    ManageSieve: use stock refresh button (bug #2923 progress)
Comment 5 users 2015-07-16 02:20:07 UTC
Changes related to this bug have been committed.
Please check latest Git and update the bug accordingly.
You can also get the patch from:
http://git.claws-mail.org/

++ ChangeLog	2015-07-16 04:20:02.787125573 +0200
http://git.claws-mail.org/?p=claws.git;a=commitdiff;h=146582502539739159668698d34a0577cccee8b2
Merge: 9b3e3b8 7c2dd3d
Author: Colin Leroy <colin@colino.net>
Date:   Thu Jul 16 04:20:02 2015 +0200

    Merge branch 'master' of file:///home/git/claws

http://git.claws-mail.org/?p=claws.git;a=commitdiff;h=7c2dd3d976388c353632ea0c3199ccdd3e214530
Author: Charles Lehner <cel@celehner.com>
Date:   Wed Jul 15 22:14:59 2015 -0400

    ManageSieve: fix manager window button spacing
    
    - Remove unused vbox
    - Move refresh button to bottom, since it is semantically different
      from the other action buttons

http://git.claws-mail.org/?p=claws.git;a=commitdiff;h=2328d40cf23a5a6c50417bc266e42c5b2b3a90bc
Author: Charles Lehner <cel@celehner.com>
Date:   Wed Jul 15 21:57:32 2015 -0400

    ManageSieve: remove Activate button
    
    A filter can be activated by selecting its radio button in the list,
    so the activate button is not needed.
    
    Bug #3469 progress

http://git.claws-mail.org/?p=claws.git;a=commitdiff;h=455254fe241f14694c85d56c6c7c3e931b8da30f
Author: Charles Lehner <cel@celehner.com>
Date:   Wed Jul 15 21:47:45 2015 -0400

    ManageSieve: save manager window size (bug #2923)

http://git.claws-mail.org/?p=claws.git;a=commitdiff;h=ae7bea75db69009e0dae20c4262e03f9ea74c8d9
Author: Charles Lehner <cel@celehner.com>
Date:   Wed Jul 15 20:31:51 2015 -0400

    ManageSieve: use stock refresh button (bug #2923 progress)
Comment 6 Charles E. Lehner 2015-07-16 02:21:56 UTC
Apologies, I meant to cite #3469, not this bug.

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