patchGNU Autoconf Archive - Patches: patch #8507, bugs in ax_perl_module_version.m4

 
 

patch #8507: bugs in ax_perl_module_version.m4

Submitted by:  Thomas Klausner <tk>
Submitted on:  Wed 06 Aug 2014 07:29:28 AM UTC  
 
Priority:  5 - Normal Status:  Done
Assigned to:  Peter Simons <simons> Open/Closed:  Closed

Add a New Comment (Rich Markup)
   

You are not logged in

Please log in, so followups can be emailed to you.

 

Thu 23 Oct 2014 10:05:24 AM UTC, comment #3: 

I've updated the syntax for shell arithmetic in 416d91b.

Peter Simons <simons>
Project AdministratorIn charge of this item.
Mon 20 Oct 2014 09:39:06 AM UTC, comment #2: 

I think you're right.
A one-line shell script fails with an arith error.

Why not use
check_args=$(($args_num % 2))

instead? I think that's more portable (but I might of course be wrong).

Thomas Klausner <tk>
Wed 15 Oct 2014 08:57:35 AM UTC, comment #1: 

I've fixed the typo in d5f4960. I'm not sure what's causing the syntax error. My guess would be that the NetBSD /bin/sh doesn't understand the "let x = ..." syntax for doing arithmetic?

Peter Simons <simons>
Project AdministratorIn charge of this item.
Wed 06 Aug 2014 07:29:28 AM UTC, original submission:  

I'm using ax_perl_module_version.m4 in one of my projects and found two issues.

The first one is that I get a syntax error for the line:

let "ckeck_args = $args_num % 2"

I don't know why that is. I'm using autoconf-2.69 and NetBSD m4.

The other bug is that this line wants to set the variable "ckeck_args" and the next line checks the value of "check_args" ('k' != 'h').

Thomas Klausner <tk>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach Files:
   
   
Comment:
   

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by simons (Posted a comment)
  • -email is unavailable- added by tk (Submitted the item)
  • -email is unavailable- added by tk
  • -email is unavailable- added by tk
  •  

     

     

    Follow 6 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2014-10-16 simons StatusReady For Test => Done
        Open/ClosedOpen => Closed
    2014-10-15 simons StatusNone => Ready For Test
        Assigned toNone => simons
    2014-08-06 tk Carbon-Copy- => Added marco gomes <mpglesi@gmail.com>
        Carbon-Copy- => Added ruben fonseca <fonseka@gmail.com>

    Back to the top


    Powered by Savane 3.6