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

 
 

patch #8507: bugs in ax_perl_module_version.m4

Submitter:  Thomas Klausner <tk>
Submitted:  Wed 06 Aug 2014 07:29:28 AM UTC
   
 
Priority:  5 - Normal Status:  Done
Assigned to:  simons Open/Closed:  Closed
* Mandatory Fields

Add a New Comment Rich Markup
   

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

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

Peter Simons <simons>
Group administrator
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>
Group administrator
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.13-758e.
    Corresponding source code