You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@httpd.apache.org by William A Rowe Jr <wr...@rowe-clan.net> on 2018/02/26 20:58:48 UTC

One more make pain point

Any objections to

--- build/mkdir.sh (revision 1825390)
+++ build/mkdir.sh (working copy)
@@ -38,7 +38,6 @@
                  continue ;;
         esac
         if test ! -d "$pathcomp"; then
-            echo "mkdir $pathcomp" 1>&2
             mkdir "$pathcomp" || errstatus=$?
         fi
         pathcomp="$pathcomp/"

That action does not rise to the alert severity of error output :-/

Or do we want to retain the notice, not redirecting to stderr?

Re: One more make pain point

Posted by William A Rowe Jr <wr...@rowe-clan.net>.
Here is the stderr vpath build result of the current 2.4.x snapshot on
Ubuntu 16.04-LTS, to illustrate my concerns;

mkdir modules
mkdir modules/aaa
mkdir modules/arch
mkdir modules/arch/win32
mkdir modules/cache
mkdir modules/core
mkdir modules/database
mkdir modules/debugging
mkdir modules/echo
mkdir modules/examples
mkdir modules/experimental
mkdir modules/filters
mkdir modules/http
mkdir modules/ldap
mkdir modules/loggers
mkdir modules/lua
configure: WARNING: *** Lua 5.3 5.2 or 5.1 library not found.
configure: WARNING: Lua 5.3 5.2 or 5.1 library is required
mkdir modules/metadata
mkdir modules/proxy
mkdir modules/session
mkdir modules/slotmem
mkdir modules/ssl
mkdir modules/test
mkdir modules/http2
mkdir modules/md
mkdir modules/proxy/balancers
mkdir server
mkdir server/mpm
mkdir server/mpm/event
mkdir server/mpm/prefork
mkdir server/mpm/worker
mkdir modules/arch/unix
mkdir modules/cluster
mkdir modules/dav
mkdir modules/dav/main
mkdir modules/generators
mkdir modules/dav/fs
mkdir modules/dav/lock
mkdir modules/mappers
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: ar: `u' modifier ignored since `D' is the default (see `U')
`u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')
ar: `u' modifier ignored since `D' is the default (see `U')


On Tue, Feb 27, 2018 at 3:34 PM, William A Rowe Jr <wr...@rowe-clan.net> wrote:
> No comment?
>
> Note that you won't see these artifacts on a same-tree build, but
> will be confronted with a slew of them on any new vpath unix build.
>
> On Mon, Feb 26, 2018 at 2:58 PM, William A Rowe Jr <wr...@rowe-clan.net> wrote:
>> Any objections to
>>
>> --- build/mkdir.sh (revision 1825390)
>> +++ build/mkdir.sh (working copy)
>> @@ -38,7 +38,6 @@
>>                   continue ;;
>>          esac
>>          if test ! -d "$pathcomp"; then
>> -            echo "mkdir $pathcomp" 1>&2
>>              mkdir "$pathcomp" || errstatus=$?
>>          fi
>>          pathcomp="$pathcomp/"
>>
>> That action does not rise to the alert severity of error output :-/
>>
>> Or do we want to retain the notice, not redirecting to stderr?

Re: One more make pain point

Posted by William A Rowe Jr <wr...@rowe-clan.net>.
No comment?

Note that you won't see these artifacts on a same-tree build, but
will be confronted with a slew of them on any new vpath unix build.

On Mon, Feb 26, 2018 at 2:58 PM, William A Rowe Jr <wr...@rowe-clan.net> wrote:
> Any objections to
>
> --- build/mkdir.sh (revision 1825390)
> +++ build/mkdir.sh (working copy)
> @@ -38,7 +38,6 @@
>                   continue ;;
>          esac
>          if test ! -d "$pathcomp"; then
> -            echo "mkdir $pathcomp" 1>&2
>              mkdir "$pathcomp" || errstatus=$?
>          fi
>          pathcomp="$pathcomp/"
>
> That action does not rise to the alert severity of error output :-/
>
> Or do we want to retain the notice, not redirecting to stderr?

Re: One more make pain point

Posted by Joe Orton <jo...@redhat.com>.
On Mon, Feb 26, 2018 at 02:58:48PM -0600, William A Rowe Jr wrote:
> Any objections to
> 
> --- build/mkdir.sh (revision 1825390)
> +++ build/mkdir.sh (working copy)
> @@ -38,7 +38,6 @@
>                   continue ;;
>          esac
>          if test ! -d "$pathcomp"; then
> -            echo "mkdir $pathcomp" 1>&2
>              mkdir "$pathcomp" || errstatus=$?
>          fi
>          pathcomp="$pathcomp/"

+1 here, seems sensible.

Regards, Joe