Skip to content

Fix fmem.h installation issue - #5

Open
maxirmx wants to merge 4 commits into
Snaipe:masterfrom
tamatebako:master
Open

Fix fmem.h installation issue#5
maxirmx wants to merge 4 commits into
Snaipe:masterfrom
tamatebako:master

Conversation

@maxirmx

@maxirmx maxirmx commented Jul 18, 2021

Copy link
Copy Markdown

Fix #3
Fix #4

cc: @ronaldtse

This is a contribution from Metanorma (GitHub), a product of Ribose (GitHub).

maxirmx and others added 4 commits July 18, 2021 12:33
- Add .github/workflows/ci.yml: matrix across Linux gcc/clang (x64+ARM64),
  macOS Intel+ARM64, Windows MSVC+MinGW, plus an ASan/UBSan static build.
- Add .github/workflows/checks.yml: trailing whitespace, CRLF, and a CMake
  configure/install sanity check.
- Add .github/workflows/release.yml: manual-trigger release PR -> merge ->
  tag v<x.y.z> -> GitHub release.
- Add .github/scripts/bump-version.sh: updates VERSION + ChangeLog.
- Delete .travis.yml and .appveyor.yml (superseded by GHA).

Packaging:
- Add VERSION file and ChangeLog.
- Add fmem.pc.in (pkg-config).
- Add cmake/fmemConfig.cmake.in so downstream can find_package(fmem)
  and link fmem::fmem.
- Add vcpkg.json + ports/fmem/ overlay.

CMakeLists.txt:
- Bump cmake_minimum_required to 3.16.
- Declare project version from VERSION; set VERSION/SOVERSION on target.
- Use target_include_directories (PUBLIC for include/ and gen/, PRIVATE
  for src/).
- Export an fmem::fmem alias target and install the export set.
- Install pkg-config file and CMake package config + version file.

README: replace Travis/AppVeyor badges with GHA badges; document
pkg-config, find_package(fmem), vcpkg, and the release workflow.

CLAUDE.md: document the new CI/packaging layout.
Criterion tags use a leading 'v' (v2.4.0+); cloning --branch 2.4.2 fails
with "Remote branch 2.4.2 not found". Switching to v2.4.3 also picks up
the latest stable release.
Criterion 2.4.x ships meson.build at the repo root, not CMakeLists.txt.
Switch the Linux install to meson + ninja using the v2.4.3 release tarball.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Wrong install statement in CmakeLists.txt Poor cmake installation support for windows.

2 participants