From: David B. <bil...@us...> - 2023-01-30 11:33:47
|
--- ** [bugs:#4094] Build warning in share: grep: warning: stray \ before #** **Status:** open **Group:** None **Created:** Mon Jan 30, 2023 11:33 AM UTC by David Billinghurst **Last Updated:** Mon Jan 30, 2023 11:33 AM UTC **Owner:** David Billinghurst There are several harmless build warnings in the share subdirectory. ~~~ grep: warning: stray \ before # ~~~ It is generated in the system call to generate make variable sharefiles. A web search didn't find any need to escape '#' in an extended grep regex. The fix is trivial. Tested with GNU egrep. I wonder if it will be a problem with other versions of egrep. ~~~ diff --git a/share/Makefile.am b/share/Makefile.am index f56124228..5681b5872 100644 --- a/share/Makefile.am +++ b/share/Makefile.am @@ -7,8 +7,8 @@ egrep -v 'CVS|Makefile(\.in|\.am)?|^\./Makefile|\.gitignore|\.gitattributes' | \ egrep -v '/fortran/' | \ egrep -v 'colnew/ex./' | \ egrep -v 'lbfgs/.*\.f' | \ -egrep -v '/\.\#' | \ -egrep -v '~|\#' | \ +egrep -v '/\.#' | \ +egrep -v '~|#' | \ sed -e 's%\./%%' | \ sort -f -d | \ sed '$$s/\\$$//' ) ~~~ --- Sent from sourceforge.net because max...@li... is subscribed to https://sourceforge.net/p/maxima/bugs/ To unsubscribe from further messages, a project admin can change settings at https://sourceforge.net/p/maxima/admin/bugs/options. Or, if this is a mailing list, you can unsubscribe from the mailing list. |
From: David B. <bil...@us...> - 2023-02-02 13:39:20
|
- **status**: open --> closed - **Comment**: Fixed by commit [66e507] --- ** [bugs:#4094] Build warning in share: grep: warning: stray \ before #** **Status:** closed **Group:** None **Created:** Mon Jan 30, 2023 11:33 AM UTC by David Billinghurst **Last Updated:** Mon Jan 30, 2023 11:33 AM UTC **Owner:** David Billinghurst There are several harmless build warnings in the share subdirectory. ~~~ grep: warning: stray \ before # ~~~ It is generated in the system call to generate make variable sharefiles. A web search didn't find any need to escape '#' in an extended grep regex. The fix is trivial. Tested with GNU egrep. I wonder if it will be a problem with other versions of egrep. ~~~ diff --git a/share/Makefile.am b/share/Makefile.am index f56124228..5681b5872 100644 --- a/share/Makefile.am +++ b/share/Makefile.am @@ -7,8 +7,8 @@ egrep -v 'CVS|Makefile(\.in|\.am)?|^\./Makefile|\.gitignore|\.gitattributes' | \ egrep -v '/fortran/' | \ egrep -v 'colnew/ex./' | \ egrep -v 'lbfgs/.*\.f' | \ -egrep -v '/\.\#' | \ -egrep -v '~|\#' | \ +egrep -v '/\.#' | \ +egrep -v '~|#' | \ sed -e 's%\./%%' | \ sort -f -d | \ sed '$$s/\\$$//' ) ~~~ --- Sent from sourceforge.net because max...@li... is subscribed to https://sourceforge.net/p/maxima/bugs/ To unsubscribe from further messages, a project admin can change settings at https://sourceforge.net/p/maxima/admin/bugs/options. Or, if this is a mailing list, you can unsubscribe from the mailing list. |
From: Robert D. <rob...@us...> - 2023-02-04 21:00:43
|
I'm running into an error in make, something about "unterminated shell command". It doesn't appear to be coming from egrep -- I tried the commands in question (`egrep -v '/\.#'` and `egrep -v '~|#'`) and they both work without error. I guess it must be either make or bash or whatever shell is invoked by make. For the record, make --version reports GNU Make 3.81, bash --version reports GNU bash, version 4.3.11(1). sh doesn't recognize --version. Looks like /bin/sh is linked to dash, and dpkg-query reports dash version is 0.5.7-4ubuntu1. This is an ancient Ubuntu (14.04) system. I'll try to point /bin/sh at a hopefully less idiosyncratic replacement. --- ** [bugs:#4094] Build warning in share: grep: warning: stray \ before #** **Status:** closed **Group:** None **Created:** Mon Jan 30, 2023 11:33 AM UTC by David Billinghurst **Last Updated:** Thu Feb 02, 2023 01:39 PM UTC **Owner:** David Billinghurst There are several harmless build warnings in the share subdirectory. ~~~ grep: warning: stray \ before # ~~~ It is generated in the system call to generate make variable sharefiles. A web search didn't find any need to escape '#' in an extended grep regex. The fix is trivial. Tested with GNU egrep. I wonder if it will be a problem with other versions of egrep. ~~~ diff --git a/share/Makefile.am b/share/Makefile.am index f56124228..5681b5872 100644 --- a/share/Makefile.am +++ b/share/Makefile.am @@ -7,8 +7,8 @@ egrep -v 'CVS|Makefile(\.in|\.am)?|^\./Makefile|\.gitignore|\.gitattributes' | \ egrep -v '/fortran/' | \ egrep -v 'colnew/ex./' | \ egrep -v 'lbfgs/.*\.f' | \ -egrep -v '/\.\#' | \ -egrep -v '~|\#' | \ +egrep -v '/\.#' | \ +egrep -v '~|#' | \ sed -e 's%\./%%' | \ sort -f -d | \ sed '$$s/\\$$//' ) ~~~ --- Sent from sourceforge.net because max...@li... is subscribed to https://sourceforge.net/p/maxima/bugs/ To unsubscribe from further messages, a project admin can change settings at https://sourceforge.net/p/maxima/admin/bugs/options. Or, if this is a mailing list, you can unsubscribe from the mailing list. |
From: Robert D. <rob...@us...> - 2023-02-04 21:53:57
|
Well, I reconfigured /bin/sh to point to /bin/bash (via sudo dpkg-reconfigure dash) and I get the same error. For the record the error message is: ``` Makefile:338: *** unterminated call to function `shell': missing `)'. ``` At this point I'm thinking the error is coming from make, which is trying to piece together the stuff to execute in a shell. I updated make to version 4.4 and now it works. Previous version 3.81 was released in 2006, so it's okay to just leave it behind. --- ** [bugs:#4094] Build warning in share: grep: warning: stray \ before #** **Status:** closed **Group:** None **Created:** Mon Jan 30, 2023 11:33 AM UTC by David Billinghurst **Last Updated:** Sat Feb 04, 2023 09:00 PM UTC **Owner:** David Billinghurst There are several harmless build warnings in the share subdirectory. ~~~ grep: warning: stray \ before # ~~~ It is generated in the system call to generate make variable sharefiles. A web search didn't find any need to escape '#' in an extended grep regex. The fix is trivial. Tested with GNU egrep. I wonder if it will be a problem with other versions of egrep. ~~~ diff --git a/share/Makefile.am b/share/Makefile.am index f56124228..5681b5872 100644 --- a/share/Makefile.am +++ b/share/Makefile.am @@ -7,8 +7,8 @@ egrep -v 'CVS|Makefile(\.in|\.am)?|^\./Makefile|\.gitignore|\.gitattributes' | \ egrep -v '/fortran/' | \ egrep -v 'colnew/ex./' | \ egrep -v 'lbfgs/.*\.f' | \ -egrep -v '/\.\#' | \ -egrep -v '~|\#' | \ +egrep -v '/\.#' | \ +egrep -v '~|#' | \ sed -e 's%\./%%' | \ sort -f -d | \ sed '$$s/\\$$//' ) ~~~ --- Sent from sourceforge.net because max...@li... is subscribed to https://sourceforge.net/p/maxima/bugs/ To unsubscribe from further messages, a project admin can change settings at https://sourceforge.net/p/maxima/admin/bugs/options. Or, if this is a mailing list, you can unsubscribe from the mailing list. |
From: David B. <bil...@us...> - 2023-02-05 01:24:47
|
I am happy to revert this patch if it is a problem. --- ** [bugs:#4094] Build warning in share: grep: warning: stray \ before #** **Status:** closed **Group:** None **Created:** Mon Jan 30, 2023 11:33 AM UTC by David Billinghurst **Last Updated:** Sat Feb 04, 2023 09:53 PM UTC **Owner:** David Billinghurst There are several harmless build warnings in the share subdirectory. ~~~ grep: warning: stray \ before # ~~~ It is generated in the system call to generate make variable sharefiles. A web search didn't find any need to escape '#' in an extended grep regex. The fix is trivial. Tested with GNU egrep. I wonder if it will be a problem with other versions of egrep. ~~~ diff --git a/share/Makefile.am b/share/Makefile.am index f56124228..5681b5872 100644 --- a/share/Makefile.am +++ b/share/Makefile.am @@ -7,8 +7,8 @@ egrep -v 'CVS|Makefile(\.in|\.am)?|^\./Makefile|\.gitignore|\.gitattributes' | \ egrep -v '/fortran/' | \ egrep -v 'colnew/ex./' | \ egrep -v 'lbfgs/.*\.f' | \ -egrep -v '/\.\#' | \ -egrep -v '~|\#' | \ +egrep -v '/\.#' | \ +egrep -v '~|#' | \ sed -e 's%\./%%' | \ sort -f -d | \ sed '$$s/\\$$//' ) ~~~ --- Sent from sourceforge.net because max...@li... is subscribed to https://sourceforge.net/p/maxima/bugs/ To unsubscribe from further messages, a project admin can change settings at https://sourceforge.net/p/maxima/admin/bugs/options. Or, if this is a mailing list, you can unsubscribe from the mailing list. |
From: David B. <bil...@us...> - 2023-02-07 10:12:54
|
Wolfgang Dautermann reverted this in commit [e00f35] with message Build system: Escape '#' in $(shell ...) command. Otherwise the build with GNU Make <= 4.2 fails, see https://stackoverflow.com/questions/53836858/ And GNU Make 4.2.1 is included in Ubuntu 20.04 so it may be in use for some years, the distribution is supported until 2025. --- ** [bugs:#4094] Build warning in share: grep: warning: stray \ before #** **Status:** closed **Group:** None **Created:** Mon Jan 30, 2023 11:33 AM UTC by David Billinghurst **Last Updated:** Sun Feb 05, 2023 01:24 AM UTC **Owner:** David Billinghurst There are several harmless build warnings in the share subdirectory. ~~~ grep: warning: stray \ before # ~~~ It is generated in the system call to generate make variable sharefiles. A web search didn't find any need to escape '#' in an extended grep regex. The fix is trivial. Tested with GNU egrep. I wonder if it will be a problem with other versions of egrep. ~~~ diff --git a/share/Makefile.am b/share/Makefile.am index f56124228..5681b5872 100644 --- a/share/Makefile.am +++ b/share/Makefile.am @@ -7,8 +7,8 @@ egrep -v 'CVS|Makefile(\.in|\.am)?|^\./Makefile|\.gitignore|\.gitattributes' | \ egrep -v '/fortran/' | \ egrep -v 'colnew/ex./' | \ egrep -v 'lbfgs/.*\.f' | \ -egrep -v '/\.\#' | \ -egrep -v '~|\#' | \ +egrep -v '/\.#' | \ +egrep -v '~|#' | \ sed -e 's%\./%%' | \ sort -f -d | \ sed '$$s/\\$$//' ) ~~~ --- Sent from sourceforge.net because max...@li... is subscribed to https://sourceforge.net/p/maxima/bugs/ To unsubscribe from further messages, a project admin can change settings at https://sourceforge.net/p/maxima/admin/bugs/options. Or, if this is a mailing list, you can unsubscribe from the mailing list. |