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

Remove the restriction on gix-url < 2.5.1 #571

Closed
pierrechevalier83 opened this issue Aug 28, 2024 · 1 comment · Fixed by #573
Closed

Remove the restriction on gix-url < 2.5.1 #571

pierrechevalier83 opened this issue Aug 28, 2024 · 1 comment · Fixed by #573

Comments

@pierrechevalier83
Copy link

gix-url was locked to be under 2.5.1 in June: 69ae6aa

The commit message states:

url crate version 2.5.1 introduces new license "Unicode-3.0" which needs to be reviewed before we can accept it

I thought I would make a tracking issue for this so I can be notified when this review has taken place and the block is removed.

In the meantime, it is complicated to depend on this and other crates like gix-url which require 2.5.2 or above for compatibility with the nightly compiler.

facebook-github-bot pushed a commit to facebookexperimental/rust-shed that referenced this issue Aug 28, 2024
Summary:
This is to make it easier to get Byron/gitoxide#1543 once it's released upstream.

There was an uncompatibility between `c2pa` which requires an older version of `url` as they want to have lawyers check the Unicode 3.0 license and `gix-url` which required latest `url` to support the nightly compiler.
I "resolved" it by removing the restriction from `c2pa` and pointing at that for now.

I also raised an issue upstream: contentauth/c2pa-rs#571 to bring it back to their awareness, so hopefully we can move off my fork soon enough.

Reviewed By: YousefSalama, dtolnay

Differential Revision: D57676392

fbshipit-source-id: fc5cabc860de9200abf3fe1145a3dbdef2a96efa
facebook-github-bot pushed a commit to facebookincubator/reindeer that referenced this issue Aug 28, 2024
Summary:
This is to make it easier to get Byron/gitoxide#1543 once it's released upstream.

There was an uncompatibility between `c2pa` which requires an older version of `url` as they want to have lawyers check the Unicode 3.0 license and `gix-url` which required latest `url` to support the nightly compiler.
I "resolved" it by removing the restriction from `c2pa` and pointing at that for now.

I also raised an issue upstream: contentauth/c2pa-rs#571 to bring it back to their awareness, so hopefully we can move off my fork soon enough.

Reviewed By: YousefSalama, dtolnay

Differential Revision: D57676392

fbshipit-source-id: fc5cabc860de9200abf3fe1145a3dbdef2a96efa
facebook-github-bot pushed a commit to facebook/dotslash that referenced this issue Aug 28, 2024
Summary:
This is to make it easier to get Byron/gitoxide#1543 once it's released upstream.

There was an uncompatibility between `c2pa` which requires an older version of `url` as they want to have lawyers check the Unicode 3.0 license and `gix-url` which required latest `url` to support the nightly compiler.
I "resolved" it by removing the restriction from `c2pa` and pointing at that for now.

I also raised an issue upstream: contentauth/c2pa-rs#571 to bring it back to their awareness, so hopefully we can move off my fork soon enough.

Reviewed By: YousefSalama, dtolnay

Differential Revision: D57676392

fbshipit-source-id: fc5cabc860de9200abf3fe1145a3dbdef2a96efa
facebook-github-bot pushed a commit to facebook/sapling that referenced this issue Aug 28, 2024
Summary:
This is to make it easier to get Byron/gitoxide#1543 once it's released upstream.

There was an uncompatibility between `c2pa` which requires an older version of `url` as they want to have lawyers check the Unicode 3.0 license and `gix-url` which required latest `url` to support the nightly compiler.
I "resolved" it by removing the restriction from `c2pa` and pointing at that for now.

I also raised an issue upstream: contentauth/c2pa-rs#571 to bring it back to their awareness, so hopefully we can move off my fork soon enough.

Reviewed By: YousefSalama, dtolnay

Differential Revision: D57676392

fbshipit-source-id: fc5cabc860de9200abf3fe1145a3dbdef2a96efa
scouten-adobe added a commit that referenced this issue Aug 29, 2024
Should fix #571. (Unfamiliar license and MSRV change added in 2.5.1 has been reverted.)
@scouten-adobe
Copy link
Collaborator

@pierrechevalier83 the license and MSRV changes in url 2.5.1 (note: url crate, not gix-url) were problematic for us and apparently many others. Just became aware that the changes were reverted, which makes things a lot easier for us. (See discussion here: servo/rust-url#937.)

We just released c2pa-rs version 0.33.4. This version now requires 2.5.2 or newer and hopefully should resolve your issues.

facebook-github-bot pushed a commit to facebook/hhvm that referenced this issue Sep 11, 2024
Summary:
This is to make it easier to get Byron/gitoxide#1543 once it's released upstream.

There was an uncompatibility between `c2pa` which requires an older version of `url` as they want to have lawyers check the Unicode 3.0 license and `gix-url` which required latest `url` to support the nightly compiler.
I "resolved" it by removing the restriction from `c2pa` and pointing at that for now.

I also raised an issue upstream: contentauth/c2pa-rs#571 to bring it back to their awareness, so hopefully we can move off my fork soon enough.

Reviewed By: YousefSalama, dtolnay

Differential Revision: D57676392

fbshipit-source-id: fc5cabc860de9200abf3fe1145a3dbdef2a96efa
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 a pull request may close this issue.

2 participants