Skip to content
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

Relax constraint on diff-lcs < 1.4 #259

Open
Annih opened this issue Jan 24, 2025 · 1 comment
Open

Relax constraint on diff-lcs < 1.4 #259

Annih opened this issue Jan 24, 2025 · 1 comment
Labels
Status: Untriaged An issue that has yet to be triaged.

Comments

@Annih
Copy link
Contributor

Annih commented Jan 24, 2025

Describe the Enhancement

In June 2020, a bug report has been created on diff-lcs about compat issue with chef-cli: halostatue/diff-lcs#65
Since the, a < 1.4.0 constraint has been added to chef-cli dependency on diff-lcs.
I want to relax this constraint and allow the use of recent versions like 1.5.1 because it fixes actual bugs in the diff computation.

It seems the only use of diff-lcs is for the chef-cli diff command; I personnally don't think it is a big deal to finally have proper diff output.
If you think

Describe the Need

Any ruby user depending on chef-cli is currently blocked on old buggy deps :(

Current Alternative

Different Gemfile ... yeah! 👎🏻

Can We Help You Implement This?

I can submit a PR to remove the constraints, but you may want a Breaking change release, I don't know, let me know!

@Annih Annih added the Status: Untriaged An issue that has yet to be triaged. label Jan 24, 2025
@Annih
Copy link
Contributor Author

Annih commented Jan 24, 2025

Ahah @nikhil2611 I see you are working on a major version release, do you think we could ship such "breaking" change?

Annih added a commit to criteo-forks/chef-cli that referenced this issue Jan 24, 2025
The version 1.3 is buggy, the only issue for chef-cli is that now the
diff command will return valid diff outputs :)

See chef#259
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status: Untriaged An issue that has yet to be triaged.
Projects
None yet
Development

No branches or pull requests

1 participant