[PATCH] ci: ignore rubygems warning in the "Documentation" job

Johannes Schindelin via GitGitGadget· Feb 9, 2020, 22:36 UTC · lore

From: Johannes Schindelin <johannes.schindelin@gmx.de>

A recent update in the Linux VM images used by Azure Pipelines surfaced a new problem in the "Documentation" job. Apparently, this warning appears 396 times on `stderr` when running `make doc`:

/usr/lib/ruby/vendor_ruby/rubygems/defaults/operating_system.rb:10: warning: constant Gem::ConfigMap is deprecated

This problem was already reported to the `rubygems` project via https://github.com/rubygems/rubygems/issues/3068.

As there is nothing Git can do about this warning, and as the "Documentation" job reports this warning as a failure, let's just silence it and move on.

Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
---
    Fix CI/PR failures in the "Documentation" job
    
    It would probably make sense to apply this to maint, too, as it will
    cause CI failures even if there is nothing actionable to be done to 
    really fix this on Git's side.

Published-As: https://github.com/gitgitgadget/git/releases/tag/pr-git-707%2Fdscho%2Fworkaround-for-rubygems-using-deprecated-component-v1
Fetch-It-Via: git fetch https://github.com/gitgitgadget/git pr-git-707/dscho/workaround-for-rubygems-using-deprecated-component-v1
Pull-Request: https://github.com/git/git/pull/707

 ci/test-documentation.sh | 1 +
 1 file changed, 1 insertion(+)

Show changes to ci/test-documentation.sh +1 −0

diff --git a/ci/test-documentation.sh b/ci/test-documentation.sh
index b3e76ef863..de41888430 100755
--- a/ci/test-documentation.sh
+++ b/ci/test-documentation.sh
@@ -7,6 +7,7 @@
 
 filter_log () {
 	sed -e '/^GIT_VERSION = /d' \
+	    -e "/constant Gem::ConfigMap is deprecated/d" \
 	    -e '/^    \* new asciidoc flags$/d' \
 	    -e '/stripped namespace before processing/d' \
 	    -e '/Attributed.*IDs for element/d' \

base-commit: d0654dc308b0ba76dd8ed7bbb33c8d8f7aacd783

-- 
gitgitgadget

Taylor Blau· Feb 9, 2020, 23:03 UTC · re: Johannes Schindelin via GitGitGadget · lore

Re: [PATCH] ci: ignore rubygems warning in the "Documentation" job

Hi Johannes,

On Sun, Feb 09, 2020 at 10:36:16PM +0000, Johannes Schindelin via GitGitGadget wrote:

Show 14 quoted lines

> From: Johannes Schindelin <johannes.schindelin@gmx.de>
>
> A recent update in the Linux VM images used by Azure Pipelines surfaced
> a new problem in the "Documentation" job. Apparently, this warning
> appears 396 times on `stderr` when running `make doc`:
>
> /usr/lib/ruby/vendor_ruby/rubygems/defaults/operating_system.rb:10: warning: constant Gem::ConfigMap is deprecated
>
> This problem was already reported to the `rubygems` project via
> https://github.com/rubygems/rubygems/issues/3068.
>
> As there is nothing Git can do about this warning, and as the
> "Documentation" job reports this warning as a failure, let's just
> silence it and move on.

Thanks for explaining, and for taking time to silence errors such as these that we can't do anything about. Everything as explained makes good sense to me.

Show 31 quoted lines

> Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
> ---
>     Fix CI/PR failures in the "Documentation" job
>
>     It would probably make sense to apply this to maint, too, as it will
>     cause CI failures even if there is nothing actionable to be done to
>     really fix this on Git's side.
>
> Published-As: https://github.com/gitgitgadget/git/releases/tag/pr-git-707%2Fdscho%2Fworkaround-for-rubygems-using-deprecated-component-v1
> Fetch-It-Via: git fetch https://github.com/gitgitgadget/git pr-git-707/dscho/workaround-for-rubygems-using-deprecated-component-v1
> Pull-Request: https://github.com/git/git/pull/707
>
>  ci/test-documentation.sh | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/ci/test-documentation.sh b/ci/test-documentation.sh
> index b3e76ef863..de41888430 100755
> --- a/ci/test-documentation.sh
> +++ b/ci/test-documentation.sh
> @@ -7,6 +7,7 @@
>
>  filter_log () {
>  	sed -e '/^GIT_VERSION = /d' \
> +	    -e "/constant Gem::ConfigMap is deprecated/d" \
>  	    -e '/^    \* new asciidoc flags$/d' \
>  	    -e '/stripped namespace before processing/d' \
>  	    -e '/Attributed.*IDs for element/d' \
>
> base-commit: d0654dc308b0ba76dd8ed7bbb33c8d8f7aacd783
> --
> gitgitgadget

Thanks, Taylor

patchci: ignore rubygems warning in the "Documentation" job