Make compile.pl / assemble.pl actually exit with maven exit code

Description

"perldoc -f system" says that we should examine $?, not $_

Acceptance / Success Criteria

None

Attachments

1
  • 17 Jan 2011, 04:54 AM

Lucidchart Diagrams

Activity

Benjamin Reed February 2, 2011 at 1:28 PM

Whups, needed to resolv this. winking face

Benjamin Reed February 2, 2011 at 1:26 PM

$? should be getting passed to that handle_errors_and_exit method (it's the return value of system()), so that's not the issue. The real issue is $_ is not the first argument to a subroutine, $_[0] is. winking face My perl was rusty.

Fixed now in 1.8 and master

[1.8 339dc61] https://opennms.atlassian.net/browse/NMS-4349#icft=NMS-4349 handle exit value properly
[master 08ebb9a] https://opennms.atlassian.net/browse/NMS-4349#icft=NMS-4349 handle exit value properly

Fixed

Details

Assignee

Reporter

Fix versions

Affects versions

Priority

PagerDuty

Created January 17, 2011 at 4:54 AM
Updated January 27, 2017 at 4:19 PM
Resolved February 2, 2011 at 1:28 PM

Flag notifications