Skip to content

Conversation

@Narfinger
Copy link
Contributor

@Narfinger Narfinger commented Jan 26, 2026

This updates the ipc-channel version.
Because the revision is fixed in servo this should be merged before servo but after ipc-channel release.

Signed-off-by: Narfinger Narfinger@users.noreply.github.com

@mrobinson mrobinson changed the title update ipc channel Upgrade ipc-channel to 0.21 Jan 27, 2026
@Narfinger Narfinger marked this pull request as ready for review January 27, 2026 08:25
@Narfinger
Copy link
Contributor Author

@mrobinson I am not sure what is up with the workflow. Does a maintainer need to do something?

@mrobinson mrobinson enabled auto-merge January 28, 2026 10:39
@mrobinson
Copy link
Member

Build failure:

Run cargo build
    Updating git repository `[https://github.com/jdm/webrender`](https://github.com/jdm/webrender%60)
    Updating crates.io index
error: failed to select a version for the requirement `ipc-channel = "^0.21"`
candidate versions found which didn't match: 0.20.2, 0.20.1, 0.20.0, ...
location searched: crates.io index
required by package `servo-media-player v0.1.0 (/home/runner/work/media/media/player)`
Error: Process completed with exit code 101.

@Narfinger
Copy link
Contributor Author

@sagudev Should then ipc-channel be bumped now or should I first have this merged with github refs?

auto-merge was automatically disabled January 28, 2026 16:38

Head branch was pushed to by a user without write access

@Narfinger
Copy link
Contributor Author

Ok I temporarily made the github revision for ipc-channel.

@sagudev
Copy link
Member

sagudev commented Jan 28, 2026

I think you should also open companion PR in servo that would use git revision of ipc channel and servo-media, then do a try ron on it and if it's all okay we should do the bump and land the PRs one by one.

@Narfinger
Copy link
Contributor Author

Companion PRs are servo/servo#42161 and this one.

@jdm
Copy link
Member

jdm commented Feb 3, 2026

This can use the published version now.

@Narfinger
Copy link
Contributor Author

Updated.

Signed-off-by: Narfinger <Narfinger@users.noreply.github.com>
Signed-off-by: Narfinger <Narfinger@users.noreply.github.com>
Signed-off-by: Josh Matthews <josh@joshmatthews.net>
@jdm jdm enabled auto-merge February 3, 2026 23:20
@jdm jdm added this pull request to the merge queue Feb 3, 2026
Merged via the queue into servo:main with commit 40e1efd Feb 3, 2026
3 checks passed
@Narfinger Narfinger deleted the ipc-channel branch February 4, 2026 08:24
github-merge-queue bot pushed a commit to servo/servo that referenced this pull request Feb 9, 2026
This updates ipc-channel.

Companion Pr in servo media: servo/media#486

This currently uses git revisions temporarily.

Signed-off-by: Narfinger <Narfinger@users.noreply.github.com>

Testing: Full try run here:
https://github.com/Narfinger/servo/actions/runs/21471578693

Signed-off-by: Narfinger <Narfinger@users.noreply.github.com>
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 this pull request may close these issues.

4 participants