CI: ugly hack to debug Windows builds in CI fail by neteler · Pull Request #2468 · OSGeo/grass

neteler · 2022-06-28T08:06:12Z

Attempt to see more (error) messages.

Related to bug #2454

Attempt to see more (error) messages. Related to bug #2454

neteler · 2022-06-28T12:23:05Z

It is weird...:

2022-06-28T08:25:24.7528731Z C:/msys64/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/12.1.0/../../../../x86_64-w64-mingw32/bin/ld.exe: warning: --export-dynamic is not supported for PE+ targets, did you mean --export-all-symbols? 2022-06-28T08:25:24.8306939Z if [ "/d/a/grass/grass/dist.x86_64-w64-mingw32/bin/d.labels.exe" != "" ] ; then GISRC=/d/a/grass/grass/dist.x86_64-w64-mingw32/demolocation/.grassrc83 GISBASE=D:/a/grass/grass/dist.x86_64-w64-mingw32 PATH="/d/a/grass/grass/dist.x86_64-w64-mingw32/bin:/d/a/grass/grass/dist.x86_64-w64-mingw32/bin:/d/a/grass/grass/dist.x86_64-w64-mingw32/scripts:$PATH" PYTHONPATH="D:/a/grass/grass/dist.x86_64-w64-mingw32/etc/python;D:/a/grass/grass/dist.x86_64-w64-mingw32/gui/wxpython;$PYTHONPATH" PATH="/d/a/grass/grass/dist.x86_64-w64-mingw32/bin:/d/a/grass/grass/dist.x86_64-w64-mingw32/bin:/d/a/grass/grass/dist.x86_64-w64-mingw32/scripts:/d/a/grass/grass/dist.x86_64-w64-mingw32/lib:/d/a/grass/grass/dist.x86_64-w64-mingw32/lib:/c/OSGeo4W/bin:/usr/bin:/mingw64/bin" LC_ALL=C LANG=C LANGUAGE=C /d/a/grass/grass/dist.x86_64-w64-mingw32/bin/d.labels.exe --html-description | grep -v '</body>\|</html>' ; fi 2022-06-28T08:25:24.8990288Z make[3]: *** [../../include/Make/Html.make:14: d.labels.tmp.html] Error 1

hellik · 2022-06-28T20:12:12Z

there are 2 issues in the configure

checking whether to use Cairo... "yes"
Package cairo was not found in the pkg-config search path.
Perhaps you should add the directory containing `cairo.pc'
to the PKG_CONFIG_PATH environment variable
Package 'cairo', required by 'virtual:world', not found
checking for location of cairo includes... /d/a/grass/grass/include
checking cairo.h usability... yes
checking cairo.h presence... yes
checking for cairo.h... yes
Package cairo was not found in the pkg-config search path.
Perhaps you should add the directory containing `cairo.pc'
to the PKG_CONFIG_PATH environment variable
checking for location of cairo library... 
checking for cairo linking flags... -L/d/a/grass/grass/mswindows/osgeo4w/lib -lcairo -lfontconfig
Package 'cairo', required by 'virtual:world', not found
checking for cairo_create... yes
checking for cairo_xlib_surface_create_with_xrender_format... no
checking for cairo_xlib_surface_get_xrender_format... no

checking whether to use FreeType... "yes"
Package freetype2 was not found in the pkg-config search path.
Perhaps you should add the directory containing `freetype2.pc'
to the PKG_CONFIG_PATH environment variable
checking for location of FreeType includes... /mingw64/include/freetype2
Package 'freetype2', required by 'virtual:world', not found
checking ft2build.h usability... yes
checking ft2build.h presence... yes
checking for ft2build.h... yes
Package freetype2 was not found in the pkg-config search path.
Perhaps you should add the directory containing `freetype2.pc'
to the PKG_CONFIG_PATH environment variable
checking for location of FreeType library... 
Package 'freetype2', required by 'virtual:world', not found
checking for FT_Init_FreeType in -lfreetype... yes

not sure what virtual:world is for.

though, config seems to be ok

  Cairo support:              yes
  DWG support:                no
  FFTW support:               yes
  FreeType support:           yes

hellik · 2022-06-28T20:34:24Z

it seems to be an issue of the build environment (results of googling the error message)

hellik · 2022-06-28T20:36:58Z

looking here in my build environment

myuser@DESKTOP-VADT8Q4 MINGW64 ~
$ pkg-config freetype2 --libs
-LC:/msys64/mingw64/lib -lfreetype

myuser@DESKTOP-VADT8Q4 MINGW64 ~
$ pkg-config freetype2 --cflags
-IC:/msys64/mingw64/include/freetype2 -IC:/msys64/mingw64/include -IC:/msys64/mingw64/include/libpng16 -IC:/msys64/mingw64/include/harfbuzz -IC:/msys64/mingw64/include/glib-2.0 -IC:/msys64/mingw64/lib/glib-2.0/include -mms-bitfields

myuser@DESKTOP-VADT8Q4 MINGW64 ~
$ pkg-config cairo --libs
-LC:/msys64/mingw64/lib -lcairo

myuser@DESKTOP-VADT8Q4 MINGW64 ~
$ pkg-config cairo --cflags
-IC:/msys64/mingw64/include/cairo -IC:/msys64/mingw64/include/lzo -IC:/msys64/mingw64/include -IC:/msys64/mingw64/include/libpng16 -IC:/msys64/mingw64/include/freetype2 -IC:/msys64/mingw64/include/harfbuzz -IC:/msys64/mingw64/include/glib-2.0 -IC:/msys64/mingw64/lib/glib-2.0/include -mms-bitfields -IC:/msys64/mingw64/include/pixman-1

myuser@DESKTOP-VADT8Q4 MINGW64 ~
$ echo $PKG_CONFIG_PATH
/mingw64/lib/pkgconfig:/mingw64/share/pkgconfig

myuser@DESKTOP-VADT8Q4 MINGW64 ~
$

no chance to do a local test build here.

not sure how to test it in the CI build environment

@ninsbl any idea?

hellik · 2022-06-28T20:39:30Z

maybe some hints in a github issue

fixed by installing mingw-w64-x86_64-pkg-config or mingw-w64-i686-pkg-config instead of pkg-config.

neteler · 2022-06-28T22:29:51Z

Here is the full log (hopefully accessible):
https://github.com/OSGeo/grass/commit/6866c3fb6b065796ef7b5de71b5d8389c55db735/checks/7088374248/logs

Here also as a ZIP file: ci_windows_mingw_errors.zip

neteler · 2022-06-29T08:29:45Z

maybe some hints in a github issue

fixed by installing mingw-w64-x86_64-pkg-config or mingw-w64-i686-pkg-config instead of pkg-config.

As far as I see this is already used - see:

grass/.github/workflows/osgeo4w.yml

Line 39 in 2af9d79

    
           run: C:\msys64\usr\bin\pacman.exe --noconfirm -S tar libintl make bison flex diffutils git dos2unix zip mingw-w64-x86_64-toolchain mingw-w64-x86_64-cairo mingw-w64-x86_64-fftw mingw-w64-x86_64-lapack mingw-w64-x86_64-pkg-config mingw-w64-x86_64-gcc mingw-w64-x86_64-ccache mingw-w64-x86_64-zlib mingw-w64-x86_64-libiconv mingw-w64-x86_64-bzip2 mingw-w64-x86_64-gettext mingw-w64-x86_64-libsystre mingw-w64-x86_64-libtre-git mingw-w64-x86_64-libwinpthread-git mingw-w64-x86_64-libpng mingw-w64-x86_64-pcre mingw-w64-x86_64-python3-six

hellik · 2022-06-29T17:08:35Z

maybe some hints in a github issue
fixed by installing mingw-w64-x86_64-pkg-config or mingw-w64-i686-pkg-config instead of pkg-config.

As far as I see this is already used - see:

grass/.github/workflows/osgeo4w.yml

Line 39 in 2af9d79

run: C:\msys64\usr\bin\pacman.exe --noconfirm -S tar libintl make bison flex diffutils git dos2unix zip mingw-w64-x86_64-toolchain mingw-w64-x86_64-cairo mingw-w64-x86_64-fftw mingw-w64-x86_64-lapack mingw-w64-x86_64-pkg-config mingw-w64-x86_64-gcc mingw-w64-x86_64-ccache mingw-w64-x86_64-zlib mingw-w64-x86_64-libiconv mingw-w64-x86_64-bzip2 mingw-w64-x86_64-gettext mingw-w64-x86_64-libsystre mingw-w64-x86_64-libtre-git mingw-w64-x86_64-libwinpthread-git mingw-w64-x86_64-libpng mingw-w64-x86_64-pcre mingw-w64-x86_64-python3-six

Though not sure mingw-w64-x86_64 is used for compiling

neteler · 2022-06-30T07:45:20Z

Though not sure mingw-w64-x86_64 is used for compiling

@hellik your remark isn't clear to me...

ninsbl · 2022-08-01T08:06:18Z

Sorry for my silence n this. For some reasons I do not receive GH notifications any more although I am watching issues...

Probably the issue is really the PKG_CONFIG_PATH., see e.g. msys2/setup-msys2#171

Will try to have a look at this the comming days...

ninsbl · 2022-08-02T13:13:04Z

The culprit is likely here:
https://github.com/OSGeo/grass/runs/7513175054?check_suite_focus=true#step:5:242

Using msys2-setup@v2 action seems to help (tests are currently running in my fork...)...

I suggest to abandon this PR in favor of #2508...

neteler · 2022-08-03T14:40:48Z

I suggest to abandon this PR in favor of #2508...

Yes, it was just a test.
Closing.

CI: ugly hack to debug Windows builds in CI fail

6866c3f

Attempt to see more (error) messages. Related to bug #2454

neteler added the CI Continuous integration label Jun 28, 2022

neteler self-assigned this Jun 28, 2022

neteler added the windows Microsoft Windows specific label Jun 28, 2022

ninsbl mentioned this pull request Aug 2, 2022

CI: introduce setup-msys2@v2 setup action #2508

Merged

neteler closed this Aug 3, 2022

neteler deleted the ci_windows_debug_only branch August 3, 2022 14:40

neteler added the invalid This doesn't seem to be a bug label Aug 3, 2022

Navigation Menu

Repository navigation

CI: ugly hack to debug Windows builds in CI fail - #2468

CI: ugly hack to debug Windows builds in CI fail#2468
neteler wants to merge 1 commit into
OSGeo:mainfrom
neteler:ci_windows_debug_only

neteler commented Jun 28, 2022

Uh oh!

neteler commented Jun 28, 2022

Uh oh!

hellik commented Jun 28, 2022

Uh oh!

hellik commented Jun 28, 2022

Uh oh!

hellik commented Jun 28, 2022

Uh oh!

hellik commented Jun 28, 2022

Uh oh!

neteler commented Jun 28, 2022

Uh oh!

neteler commented Jun 29, 2022

Uh oh!

hellik commented Jun 29, 2022

Uh oh!

neteler commented Jun 30, 2022 •

edited

Loading

Uh oh!

ninsbl commented Aug 1, 2022

Uh oh!

ninsbl commented Aug 2, 2022

Uh oh!

neteler commented Aug 3, 2022

Uh oh!

Reviewers

Uh oh!

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Conversation

neteler commented Jun 28, 2022

Uh oh!

neteler commented Jun 28, 2022

Uh oh!

hellik commented Jun 28, 2022

Uh oh!

hellik commented Jun 28, 2022

Uh oh!

hellik commented Jun 28, 2022

Uh oh!

hellik commented Jun 28, 2022

Uh oh!

neteler commented Jun 28, 2022

Uh oh!

neteler commented Jun 29, 2022

Uh oh!

hellik commented Jun 29, 2022

Uh oh!

neteler commented Jun 30, 2022 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

ninsbl commented Aug 1, 2022

Uh oh!

ninsbl commented Aug 2, 2022

Uh oh!

neteler commented Aug 3, 2022

Uh oh!

Reviewers

Uh oh!

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

neteler commented Jun 30, 2022 •

edited

Loading