8000 Remove autotools & modernise build CI/CD by ThomasAdam · Pull Request #1191 · fvwmorg/fvwm3 · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Remove autotools & modernise build CI/CD #1191

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 4 commits into from
May 15, 2025
Merged

Conversation

ThomasAdam
Copy link
Member
@ThomasAdam ThomasAdam commented May 10, 2025

This PR addresses the change to the infrastructure of fvwm3, removing autotools and asserting meson as the primary/only build-tool to use.

This hasn't been mandatory -- it's gone through an extensive consoltation period over many months to get to this point -- as individual commits will attest to.

In addition to this change though, there is a wider implication around changing the buildsystem fvwm3 uses to make use of meson.

In doing so, this change has also meant automating fvwm3's release, removing key persons as a bottle-neck to that.

The status of the release needs to be kept, but it can no
longer be a part of ./configure as that is being removed.
@ThomasAdam ThomasAdam added this to the 1.1.3 milestone May 10, 2025
@ThomasAdam ThomasAdam added the area:build Relates to compiling/buildsystem of fvwm label May 10, 2025
@ThomasAdam ThomasAdam added this to FVWM3 May 10, 2025
@github-project-automation github-project-automation bot moved this to PRs in FVWM3 May 10, 2025
@ThomasAdam ThomasAdam force-pushed the ta/remove-autotools branch 25 times, most recently from f8249b8 to b505d4b Compare May 10, 2025 15:36
@ThomasAdam ThomasAdam force-pushed the ta/remove-autotools branch 19 times, most recently from b107d5d to e7d2810 Compare May 14, 2025 17:09
@ThomasAdam ThomasAdam changed the title ta/remove autotools Remove autotools & modernise build CI/CD May 14, 2025
@ThomasAdam ThomasAdam requested a review from somiaj May 14, 2025 17:47
Copy link
Collaborator
@somiaj somiaj left a comment

Choose a reason for hiding this comment

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

Looks good. I didn't test the release workflow, but I did testing building a debian package, and the debian package not only builds, I can build it without specifically telling debian helper to use meson, since the auto 8000 tool files are no longer present.

Rewrite the release process such that releases can be fully automated.
@ThomasAdam ThomasAdam force-pushed the ta/remove-autotools branch from e7d2810 to 26182df Compare May 15, 2025 07:23
This is mostly relevant for release tarballs, but developer
documentation doesn't get shipped in a release.  However, installation
instructions are still required.

Therefore, make the INSTALL.md file part of the root of the project.

Fixes #1194
@github-actions github-actions bot merged commit be79cdf into main May 15, 2025
9 checks passed
@github-actions github-actions bot deleted the ta/remove-autotools branch May 15, 2025 17:47
@github-project-automation github-project-automation bot moved this from PRs to Done in FVWM3 May 15, 2025
@ThomasAdam ThomasAdam restored the ta/remove-autotools branch May 15, 2025 17:52
netbsd-srcmastr pushed a commit to NetBSD/pkgsrc that referenced this pull request Jun 7, 2025
# changes

### Enhancements
* FvwmRearrange: Use -on_screen to tile/cascade all windows on the monitor. by @pghvlaans in fvwmorg/fvwm3#1170
* FvwmRearrange: Add -win_cmd COMMAND option. by @somiaj in fvwmorg/fvwm3#1179
* expand.c: Add "maximum" and "minimum" to math. by @pghvlaans in fvwmorg/fvwm3#1176

### Other Changes
* build: macos: fix path to fribidi by @ThomasAdam in fvwmorg/fvwm3#1174
* fvwm-convert-2.6: fix syntax error by @ThomasAdam in fvwmorg/fvwm3#1177
* style: remove superfluous debug by @ThomasAdam in fvwmorg/fvwm3#1181
* Remove autotools & modernise build CI/CD by @ThomasAdam in fvwmorg/fvwm3#1191
* build: remove go-1.14.0 check by @ThomasAdam in fvwmorg/fvwm3#1195
* Fix crash in FvwmPager and clarify Balloons option. by @somiaj in fvwmorg/fvwm3#1197
* build: modify release version script by @ThomasAdam in fvwmorg/fvwm3#1201
* meson: do a link test for `locale_charset()` by @Kangie in fvwmorg/fvwm3#1203
* Small RandR fixes by @ThomasAdam in fvwmorg/fvwm3#1207
* setup_visible_names: don't ignore when both set by @ThomasAdam in fvwmorg/fvwm3#1208
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:build Relates to compiling/buildsystem of fvwm
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants
0