Skip to content

Added function for autotools configure to cat config.log in case of failure - #2602

Open
craigcomstock wants to merge 2 commits into
cfengine:masterfrom
craigcomstock:ent-14916/master
Open

Added function for autotools configure to cat config.log in case of failure#2602
craigcomstock wants to merge 2 commits into
cfengine:masterfrom
craigcomstock:ent-14916/master

Conversation

@craigcomstock

@craigcomstock craigcomstock commented Nov 2, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Ticket: ENT-14916
Changelog: none

with cfengine/core#6497

@craigcomstock

craigcomstock commented Nov 2, 2026

Copy link
Copy Markdown
Contributor Author

@cf-bottom jenkins no tests

@craigcomstock craigcomstock mentioned this pull request Nov 2, 2026
Closed
@cf-bottom

cf-bottom commented Nov 2, 2026

Copy link
Copy Markdown

Alright, I triggered a build:

Build Status

[NO TESTS]

Jenkins: https://ci.cfengine.com/job/pr-pipeline/14984/

Packages: http://buildcache.cfengine.com/packages/testing-pr/jenkins-pr-pipeline-14984/

@craigcomstock

craigcomstock commented Nov 3, 2026

Copy link
Copy Markdown
Contributor Author

@cf-bottom jenkins no tests

@cf-bottom

cf-bottom commented Nov 3, 2026

Copy link
Copy Markdown

Alright, I triggered a build:

Build Status

[NO TESTS]

Jenkins: https://ci.cfengine.com/job/pr-pipeline/14988/

Packages: http://buildcache.cfengine.com/packages/testing-pr/jenkins-pr-pipeline-14988/

@craigcomstock

craigcomstock commented Nov 3, 2026

Copy link
Copy Markdown
Contributor Author

@cf-bottom jenkins no tests

@cf-bottom

cf-bottom commented Nov 3, 2026

Copy link
Copy Markdown

Alright, I triggered a build:

Build Status

[NO TESTS]

Jenkins: https://ci.cfengine.com/job/pr-pipeline/14994/

Packages: http://buildcache.cfengine.com/packages/testing-pr/jenkins-pr-pipeline-14994/

@craigcomstock
craigcomstock marked this pull request as ready for review November 3, 2026 22:33
@craigcomstock

craigcomstock commented Nov 3, 2026

Copy link
Copy Markdown
Contributor Author

I will revert the GH workflow disablements and squash commits tomorrow.

larsewi
larsewi approved these changes Nov 8, 2026
View reviewed changes
Comment thread build-scripts/functions Resolved
@craigcomstock

craigcomstock commented Nov 10, 2026

Copy link
Copy Markdown
Contributor Author

@cf-bottom jenkins no tests

@cf-bottom

cf-bottom commented Nov 10, 2026

Copy link
Copy Markdown

Sure, I triggered a build:

Build Status

[NO TESTS]

Jenkins: https://ci.cfengine.com/job/pr-pipeline/15066/

Packages: http://buildcache.cfengine.com/packages/testing-pr/jenkins-pr-pipeline-15066/

@craigcomstock

craigcomstock commented Nov 10, 2026

Copy link
Copy Markdown
Contributor Author

The happy case seems OK.

13:13:54 configure: debug: running configure on core repo...
13:14:54 configure: debug: running configure on enterprise repo...
13:15:01 configure: debug: running configure on nova repo...
13:15:06 configure: debug: running configure on masterfiles repo...

I will add a breaking core PR to this and see how it fares then.

@craigcomstock
Added function for autotools configure to cat config.log in case of f…
6848f22 
…ailure

Removed $OPTS from configure script. It has been there since the initial commit but seems unused so will remove as it simplifies this script.

Ticket: ENT-14916
Changelog: none
@craigcomstock craigcomstock mentioned this pull request Nov 10, 2026
Draft
@craigcomstock

craigcomstock commented Nov 10, 2026

Copy link
Copy Markdown
Contributor Author

Build Status

@craigcomstock

craigcomstock commented Nov 11, 2026

Copy link
Copy Markdown
Contributor Author

Build Status

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@larsewi larsewi larsewi approved these changes
@aleksandrychev aleksandrychev Awaiting requested review from aleksandrychev

Assignees

No one assigned

Labels

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@craigcomstock @cf-bottom @larsewi