Skip to content
This repository has been archived by the owner on Apr 9, 2023. It is now read-only.

Upgrade to nom 6 #20

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Upgrade to nom 6 #20

wants to merge 2 commits into from

Conversation

mbrubeck
Copy link

@mbrubeck mbrubeck commented Jun 6, 2020

Fixes #15. Closes #19.

@mbrubeck mbrubeck force-pushed the nom5 branch 2 times, most recently from 5132e9d to 21eae5f Compare June 6, 2020 19:04
@mbrubeck
Copy link
Author

mbrubeck commented Jun 6, 2020

I also have this branch which merges in the latest changes from master, but does not compile (because of the existing errors on master):

master...mbrubeck:nom5-rebase-wip

@mbrubeck
Copy link
Author

This change is included in my stripped-down fork of this crate:

https://crates.io/crates/tree_magic_mini

@audunhalland
Copy link

nom 6 has been released.

@mbrubeck
Copy link
Author

mbrubeck commented Jan 9, 2021

Thanks. I've updated this branch to use nom 6. I also applied this change to my fork tree_magic_mini.

@mbrubeck mbrubeck changed the title Upgrade to nom 5.1 Upgrade to nom 6 Jan 9, 2021
emixa-d added a commit to emixa-d/antioxidant-fallback that referenced this pull request Jun 29, 2022
…bundle.

The fork is used for being compatible with the new rust-nom, see
<aahancoc/tree_magic#20>.

- new package definition, using the fork
- new replacement
- disable feature for avoiding code that expects bundling
- add phase to make rust-tree-magic-mini find the unbundled magic data
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.

nom v4
2 participants