finally removing gh links from sdl-cli #6

Merged
jahway603 merged 2 commits from jahway603/silentdragonlite-cli:dev into dev-2021 3 years ago
Collaborator
There is no content yet.
jahway603 added 2 commits 3 years ago
jahway603 merged commit 8d785d708a into dev-2021 3 years ago
Owner

@jahway603 have you tested that this works correctly? It's not clear to me that it does.

If you go to https://git.hush.is/hush/librustzcash.git?rev=1a0204113d487cdaaf183c2967010e5214ff9e37#1a0204113d487cdaaf183c2967010e5214ff9e37 it does not show the 1a020... commit, because Gitea does not seem to interpret the ?rev= URL param the same way as Github.

Our stuff definitely depends on this special comming and many things will break if it's not used.

@jahway603 have you tested that this works correctly? It's not clear to me that it does. If you go to https://git.hush.is/hush/librustzcash.git?rev=1a0204113d487cdaaf183c2967010e5214ff9e37#1a0204113d487cdaaf183c2967010e5214ff9e37 it does not show the 1a020... commit, because Gitea does not seem to interpret the `?rev=` URL param the same way as Github. Our stuff definitely depends on this special comming and many things will break if it's not used.
Poster
Collaborator

Reverted back to GH links with their special "?rev" in this pull request here

Possible solutions

  • Incorp the right commits from these 100+ located here

  • Figure out how to do special commit on Gitea with their "?rev"-equivalent

Reverted back to GH links with their special "?rev" [in this pull request here](https://git.hush.is/hush/silentdragonlite-cli/pulls/9) **Possible solutions** * Incorp the right commits from [these 100+ located here](https://git.hush.is/hush/librustzcash/commits/commit/c62503416377eb31fb74cbbcd4db91206df561fb) * Figure out how to do special commit on Gitea with their "?rev"-equivalent
Owner

@jahway603 another way to solve this would be to tell Cargo.lock to use the sdl branch which that commit corresponds to

@jahway603 another way to solve this would be to tell Cargo.lock to use the `sdl` branch which that commit corresponds to
Owner

@jahway603 when I look at https://github.com/MyHush/librustzcash?rev=1a0204113d487cdaaf183c2967010e5214ff9e37 it also does not show me that commit in the web interface. I am not sure if that rev= param is only used by Cargo maybe?

@jahway603 when I look at https://github.com/MyHush/librustzcash?rev=1a0204113d487cdaaf183c2967010e5214ff9e37 it also does not show me that commit in the web interface. I am not sure if that `rev=` param is only used by Cargo maybe?
The pull request has been merged as 8d785d708a.
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Loading…
There is no content yet.