Skip to content

Updated dependencies (master)#2340

Draft
github-actions[bot] wants to merge 15 commits into
masterfrom
update-dependencies-action-master-1783929879
Draft

Updated dependencies (master)#2340
github-actions[bot] wants to merge 15 commits into
masterfrom
update-dependencies-action-master-1783929879

Conversation

@github-actions

@github-actions github-actions Bot commented Jul 13, 2026

Copy link
Copy Markdown
Contributor

Automated dependency updates for the master branch

together: cfengine/core#6235

@larsewi

larsewi commented Jul 13, 2026

Copy link
Copy Markdown
Contributor

@cf-bottom Jenkins please :)

@cf-bottom

Copy link
Copy Markdown

@larsewi

larsewi commented Jul 13, 2026

Copy link
Copy Markdown
Contributor

@cf-bottom Jenkins please :)

@cf-bottom

Copy link
Copy Markdown

@olehermanse

olehermanse commented Jul 13, 2026

Copy link
Copy Markdown
Member

will rebase

@olehermanse olehermanse reopened this Jul 13, 2026
@olehermanse olehermanse force-pushed the update-dependencies-action-master-1783929879 branch from 0d4779f to 60f655b Compare July 13, 2026 18:03
@cfengine cfengine deleted a comment from olehermanse Jul 13, 2026
Ticket: none
Changelog: none
@craigcomstock craigcomstock force-pushed the update-dependencies-action-master-1783929879 branch from 993418e to f882a56 Compare July 13, 2026 19:45
@craigcomstock

Copy link
Copy Markdown
Contributor

@cf-bottom jenkins please

@cf-bottom

Copy link
Copy Markdown

Adjusting core code to work with a dynamic max key size in lmdb is a larger effort.

Ticket: ENT-14295
Changelog: none
@craigcomstock

Copy link
Copy Markdown
Contributor

@cf-bottom jenkins please

@cf-bottom

Copy link
Copy Markdown

@aleksandrychev aleksandrychev self-assigned this Jul 14, 2026
Signed-off-by: Ihor Aleksandrychiev <ihor.aleksandrychiev@northern.tech>
@aleksandrychev

Copy link
Copy Markdown
Contributor

@cf-bottom jenkins please

@cf-bottom

Copy link
Copy Markdown

@aleksandrychev aleksandrychev marked this pull request as draft July 14, 2026 10:27
@aleksandrychev

Copy link
Copy Markdown
Contributor

windows only - Build Status before running full pipeline

@aleksandrychev

Copy link
Copy Markdown
Contributor

PACKAGES_x86_64_mingw successfully built

@cf-bottom please jenkins

@cf-bottom

Copy link
Copy Markdown

From 0000000000000000000000000000000000000000 Mon Sep 17 00:00:00 2001
From: Ihor Aleksandrychiev <igor.aleksandrychev@northern.tech>
Date: Tue, 14 Jul 2026 15:19:32 +0300
Subject: [PATCH] Fix Windows/mingw LARGE_INTEGER usage in mdb.c

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this should likely have an upstream bug number so we can track when to remove this patch.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i looked at https://bugs.openldap.org/buglist.cgi?bug_status=__open__&content=windows&list_id=7358&order=Importance&query_format=specific (requires account I think) and didn't see any issues, but maybe there is one and maybe you need to create one.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

right, https://bugs.openldap.org/show_bug.cgi?id=10539#c1 in the next release should be fixed. added comment


as_fn_error $? "Could not find pthreads library" "$LINENO" 5

+fi

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we should not be patching the configure script, but rather configure.ac and in this case Makefile.am where I recently added -ldl.

It's a bit of a mess as we add the autotools stuff and then in either patch 0005 or 0009 we add files like this configure which we should really NOT include, preferring to run autoconf/etc instead.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

good catch, moved the -ldl fix into the autotools source. 0012 now patches configure.ac and Makefile.am. 0013 carries the regenerated configure/Makefile.in, same split as 0009/0010.

aleksandrychev and others added 2 commits July 14, 2026 21:54
lmdb 1.0.0 added incremental dump/load and MDB_RPAGE_CACHE remapping
code that treats the Windows LARGE_INTEGER union as a scalar integer,
which fails to compile with x86_64-w64-mingw32-gcc. Add a patch that
accesses the .QuadPart member on Windows so offset arithmetic and
SetFilePointerEx() use the correct type. POSIX builds are unaffected.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
lmdb 1.0.0's autotools source linked liblmdb (and, transitively, the
tools) against -ldl unconditionally, via both liblmdb_la_LDFLAGS and
AM_LDFLAGS in Makefile.am. libdl does not exist on Windows, so the mingw
cross build failed with 'cannot find -ldl' when linking mdb_stat.exe.
dlopen() is only used by module.c on Unix; Windows uses LoadLibrary().

Detect dlopen via AC_CHECK_LIB in configure.ac (adding -ldl to LIBS only
when it links) and drop the hard-coded -ldl from Makefile.am. On Unix
-ldl still flows into every link through $(LIBS); on Windows it is simply
omitted. Split into a source patch (configure.ac, Makefile.am) and the
regenerated files (configure, Makefile.in), matching the 0009/0010 split.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@aleksandrychev aleksandrychev force-pushed the update-dependencies-action-master-1783929879 branch from b4b906a to 679f7b5 Compare July 14, 2026 18:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

5 participants