raulcd commented on PR #47864:
URL: https://github.com/apache/arrow/pull/47864#issuecomment-3486773286

   > (I'm trying this on local but it doesn't work yet. Sorry...)
   
   As I am unable to fix the meson built pkgconfig files on the case of running 
reprotest with the `build_path` variation inside the docker container from the 
rake task I tried the proposed approach of running reprotest on the host.
   I applied the suggested patch and I run reprotest locally. I had to modify 
some minor things (i.e, the path for artifacts because glob pattern doesn't 
seem to work) and I decided to disable some variations as I was getting some 
failures due to disorderfs or SSL expiration (due to time variation). I ended 
up running:
   > reprotest -vv --vary=-kernel,-fileordering,-domain_host,-build_path,-time 
--store-dir=/tmp/repro-logs -c "BUILD_DIR=/tmp/arrow-build rake -C 
dev/tasks/linux-packages --trace apt:build APT_TARGETS=debian-trixie" . \ 
"dev/tasks/linux-packages/*/apt/repositories/*"
   
   which is able to run both variations but diffoscope takes forever to finish. 
Running diffoscope manually showed some variations due to this change:
   ```diff
   @@ -67,6 +68,8 @@ if which ccache > /dev/null 2>&1; then
        debuild_options+=(--prepend-path=/usr/lib/ccache)
      fi
    fi
   +build_dir=$(mktemp --directory --tmpdir="${build_root_dir}" package.XXXXX)
   +run pushd "${build_dir}"
    run cp /host/tmp/${PACKAGE}-${VERSION}.tar.gz \
      ${PACKAGE}_${VERSION}.orig.tar.gz
    run tar xfz ${PACKAGE}_${VERSION}.orig.tar.gz
   ```
   like:
   ```diff
   │ │ │ │ @@ -63360,15 +63360,15 @@
   │ │ │ │  N6google5cloud5v2_227Options4DataINS1_23UserAgentProductsOptionEEE
   │ │ │ │  N6google5cloud5v2_228internal20UniverseDomainOptionE
   │ │ │ │  N6google5cloud5v2_2218RuntimeStatusErrorE
   │ │ │ │  
St23_Sp_counted_ptr_inplaceIN6google5cloud5v2_228internal14StdClogBackendESaIvELN9__gnu_cxx12_Lock_policyE2EE
   │ │ │ │  
St23_Sp_counted_ptr_inplaceIN6google5cloud5v2_228internal21CircularBufferBackendESaIvELN9__gnu_cxx12_Lock_policyE2EE
   │ │ │ │  
St23_Sp_counted_ptr_inplaceIN6google5cloud5v2_228internal30PerThreadCircularBufferBackendESaIvELN9__gnu_cxx12_Lock_policyE2EE
   │ │ │ │  
St23_Sp_counted_ptr_inplaceIKN6google5cloud5v2_227OptionsESaIvELN9__gnu_cxx12_Lock_policyE2EE
   │ │ │ │ --g -O2 
-ffile-prefix-map=/build/package.gp79U/apache-arrow-23.0.0~dev20251104=. 
-fstack-protector-strong -fstack-clash-protection -Wformat 
-Werror=format-security -fcf-protection -D_FORTIFY_SOURCE=2 
-fdiagnostics-color=always -fPIC -O2 -g -DNDEBUG -ftree-vectorize -ggdb
   │ │ │ │ +-g -O2 
-ffile-prefix-map=/build/package.JQ1Hg/apache-arrow-23.0.0~dev20251104=. 
-fstack-protector-strong -fstack-clash-protection -Wformat 
-Werror=format-security -fcf-protection -D_FORTIFY_SOURCE=2 
-fdiagnostics-color=always -fPIC -O2 -g -DNDEBUG -ftree-vectorize -ggdb
   │ │ │ │  N6google5cloud5v2_2210LogBackendE
   │ │ │ │  N6google5cloud5v2_228internal14StdClogBackendE
   │ │ │ │  N6google5cloud5v2_228internal21CircularBufferBackendE
   │ │ │ │  N6google5cloud5v2_228internal30PerThreadCircularBufferBackendE
   │ │ │ │  N3Aws4Auth32STSAssumeRoleCredentialsProviderE
   │ │ │ │  
St23_Sp_counted_ptr_inplaceIN3Aws3STS9STSClientESaIS2_ELN9__gnu_cxx12_Lock_policyE2EE
   │ │ │ │  N3Aws8Internal8Endpoint18EndpointAuthSchemeE
   ```
   or
   ```diff
   │ │ ├── ./usr/lib/x86_64-linux-gnu/pkgconfig/gandiva-glib.pc
   │ │ │ @@ -7,9 +7,9 @@
   │ │ │  
   │ │ │  Name: Apache Arrow Gandiva GLib
   │ │ │  Description: C API for Apache Arrow Gandiva based on GLib
   │ │ │  Version: 23.0.0-SNAPSHOT
   │ │ │  Requires: gandiva, arrow-glib
   │ │ │  Requires.private: gobject-2.0
   │ │ │  Libs: -L${libdir} -lgandiva-glib
   │ │ │ -Libs.private: 
/build/package.gp79U/apache-arrow-23.0.0~dev20251104/c_glib/../cpp_build/relwithdebinfo/libgandiva.so
 
/build/package.gp79U/apache-arrow-23.0.0~dev20251104/c_glib/../cpp_build/relwithdebinfo/libarrow_acero.so
 
/build/package.gp79U/apache-arrow-23.0.0~dev20251104/c_glib/../cpp_build/relwithdebinfo/libarrow_compute.so
 
/build/package.gp79U/apache-arrow-23.0.0~dev20251104/c_glib/../cpp_build/relwithdebinfo/libarrow.so
 /usr/lib/x86_64-linux-gnu/libgio-2.0.so
   │ │ │ +Libs.private: 
/build/package.JQ1Hg/apache-arrow-23.0.0~dev20251104/c_glib/../cpp_build/relwithdebinfo/libgandiva.so
 
/build/package.JQ1Hg/apache-arrow-23.0.0~dev20251104/c_glib/../cpp_build/relwithdebinfo/libarrow_acero.so
 
/build/package.JQ1Hg/apache-arrow-23.0.0~dev20251104/c_glib/../cpp_build/relwithdebinfo/libarrow_compute.so
 
/build/package.JQ1Hg/apache-arrow-23.0.0~dev20251104/c_glib/../cpp_build/relwithdebinfo/libarrow.so
 /usr/lib/x86_64-linux-gnu/libgio-2.0.so
   │ │ │  Cflags: -I${includedir}
   ```
   
   I'll try having a fixed `build_dir` and will start to work towards trying to 
remove variation skips. Just wanted to share progress on where am I.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to