Skip to content
This repository was archived by the owner on Dec 8, 2024. It is now read-only.

Compatibility with latest libfyaml#4

Open
vaab wants to merge 11 commits into
dtolnay:masterfrom
0k:master
Open

Compatibility with latest libfyaml#4
vaab wants to merge 11 commits into
dtolnay:masterfrom
0k:master

Conversation

@vaab

@vaab vaab commented Jan 28, 2024

Copy link
Copy Markdown

I'm unsure if you are interested in these changes. Here they are:

  • dynamically compute a sane VERSION from git tags (or git describe if off tag) for libfyaml
  • re-enable the fy_library_version() binding using VERSION
  • dynamically compute the vargs blocklist functions to match the different sets between v0.8, v0.9, and latest master
  • dynamically add include paths amongst a list of candidate depending on existence
  • propose in the last commit to update libfyaml to latest master
  • add some cargo:rerun-if-changed=.. on C files to allow for seamless hacking on libfyaml's code

The aim here is to facilitate integration of future changes in libfyaml. I felt the need also to be able to go back and forth between the current version you had (v0.8) and the last official version (v0.9) and also be able to test with latest master. All 3 of these versions are compiling on my side.

Also, I'm pretty new to Rust, and am very happy of any feedbacks.
Many thanks for the bindings. They are already in use in my projects.

This helps to smoothly compile between different version of
``libfyaml``.
@vaab vaab force-pushed the master branch 2 times, most recently from f931ff2 to 94c92e2 Compare January 29, 2024 15:46
vaab added 7 commits January 19, 2026 05:14
We can enable back ``fy_library_version``.
Add ``libc::iovec`` import required by v0.9.3's new ``sys/uio.h``
dependency. Update version to 0.2.7+fy0.9.3.

Add CHANGELOG.md documenting upstream changes including new composer
and document builder APIs, bug fixes, and performance improvements.
- Remove redundant ``.to_string()`` in format args
- Remove needless borrow in regex match
- Simplify closure to function reference for ``PathBuf::from``
- Allow ``unnecessary_transmutes`` in bindgen-generated code
- Rename crate from ``libfyaml-sys`` to ``fyaml-sys`` as maintained fork
- Flatten repo structure: crate now at root instead of subdirectory
- Move ``libfyaml`` submodule to root level
- Update all metadata to credit dtolnay as original author
- Update repository URL to ``0k/fyaml-sys``

This creates a clean separation from the archived upstream while
preserving proper attribution.
vaab added 3 commits January 19, 2026 10:32
- Update ``libfyaml`` submodule 16 commits past v0.9.3
- Bump version to ``0.1.1-alpha.1+fy0.9.3-16-ged4720d``
- Add David Tolnay as co-author in crate metadata
- Restructure ``CHANGELOG.md`` with separate fyaml-sys / libfyaml-sys
  sections and document upstream changes
- Remove ``ci.yml`` (depended on ``dtolnay/.github`` infra)
- Remove ``FUNDING.yml`` (credited ``dtolnay``)
- Add copyright header to ``LICENSE-MIT``
- Add ``exclude`` in ``Cargo.toml`` for non-release files
- Fix ``rust-version`` from 1.63 to 1.70 (``bindgen`` 0.69 requires it)
- Add ``.pkg`` to ``.gitignore``
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant