Skip to content

Postcard remove default embedded dependencies#440

Merged
sagudev merged 1 commit intoservo:mainfrom
Narfinger:postcard-feature
Jan 27, 2026
Merged

Postcard remove default embedded dependencies#440
sagudev merged 1 commit intoservo:mainfrom
Narfinger:postcard-feature

Conversation

@Narfinger
Copy link
Contributor

@Narfinger Narfinger commented Jan 26, 2026

Postcard is aimed at embedded devices, so has things like heapless in it. We can save these dependencies for our purposes and just rely on the std feature flag.

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

@Narfinger Narfinger changed the title Postcard is aimed at embedded devices so has heapless as standard. We don't need this in our case and can remove it saving a couple of dependencies. Postcard remove deffault embedded dependencies Jan 26, 2026
don't need this in our case and can remove it saving a couple of
dependencies.

Signed-off-by: Narfinger <Narfinger@users.noreply.github.com>
@Narfinger Narfinger marked this pull request as ready for review January 26, 2026 14:21
@Narfinger Narfinger changed the title Postcard remove deffault embedded dependencies Postcard remove default embedded dependencies Jan 26, 2026
@glyn
Copy link
Contributor

glyn commented Jan 26, 2026

@Narfinger: the CI failure is opaque to me. Any ideas?

@Narfinger
Copy link
Contributor Author

Let me try to restart it and see how it goes.

@Narfinger
Copy link
Contributor Author

Still errors. I am not sure why.

@glyn
Copy link
Contributor

glyn commented Jan 26, 2026

Still errors. I am not sure why.

Since there are no details, I suggest running the tests on a Windows box/VM (shudder) or else commenting out the failing tests and seeing if the other Windows jobs give us some clues.

@glyn
Copy link
Contributor

glyn commented Jan 26, 2026

Another option might be to grab the logs using the gh CLI as per https://github.com/orgs/community/discussions/53164#discussioncomment-10569630 (also the whole discussion is about a similar issue).

(I wonder if the transitive dependencies on Windows turn out to be a problem?)

@Narfinger
Copy link
Contributor Author

Ok the jobs are doing things now. Looked like a fluke.

@sagudev sagudev added this pull request to the merge queue Jan 27, 2026
Merged via the queue into servo:main with commit 1cfbb8d Jan 27, 2026
66 of 100 checks passed
@Narfinger Narfinger deleted the postcard-feature branch January 27, 2026 08:44
@glyn
Copy link
Contributor

glyn commented Jan 27, 2026

What do we make of the outstanding CI failure(s)? Do we write them off and attribute them to some kind of infrastructure glitch?

@sagudev
Copy link
Member

sagudev commented Jan 27, 2026

What do we make of the outstanding CI failure(s)? Do we write them off and attribute them to some kind of infrastructure glitch?

Yes

glyn added a commit to glyn/ipc-channel-mux that referenced this pull request Feb 5, 2026
Since ipc-channel switched, do the same here.

Rationale for ipc-channel switching:

servo/ipc-channel#432 (comment)

See also:

servo/ipc-channel#440
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