[asterisk-dev] [Code Review] 2304: Clean up Makefile "warning" clutter when makeopts doesn't exist

wdoekes reviewboard at asterisk.org
Mon Apr 8 13:51:55 CDT 2013


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/2304/
-----------------------------------------------------------

(Updated April 8, 2013, 6:51 p.m.)


Status
------

This change has been marked as submitted.


Review request for Asterisk Developers and Jason Parker.


Repository: Asterisk


Description
-------

When there is no makeopts (distclean removes it), we get ugly errors when running make.

$ make
build_tools/make_version: 183: URL: not found
build_tools/make_version: 183: {print $2;}: not found
svn: Write error: Broken pipe
/bin/sh: Illegal option - 
/bin/sh: Illegal option - 
...
****
**** The configure script must be executed before running 'make'.
****               Please run "./configure".
****

Those ugly errors can be avoided in the majority of case by providing defaults for AWK and GREP. That's what this patch does.

This means that you can now run make distclean twice and get output without ugly errors.


P.S. @admins: https://reviewboard.asterisk.org/r/2303/ yields a Django error and is uneditable because I accidentally called qwell "Jason Parker".


Diffs
-----

  /trunk/Makefile 383540 
  /trunk/build_tools/make_buildopts_h 383540 
  /trunk/build_tools/make_linker_version_script 383540 
  /trunk/build_tools/make_version 383540 
  /trunk/build_tools/mkpkgconfig 383540 

Diff: https://reviewboard.asterisk.org/r/2304/diff/


Testing
-------

It works if awk en grep are in your path. If they aren't, the errors will at least be more readable (missing grep instead of "Illegal option").

AWK and GREP will still get overridden by makeopts if it exists, so no loss of functionality there.


Thanks,

wdoekes

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20130408/1faa07cc/attachment.htm>


More information about the asterisk-dev mailing list