Skip to content
This repository has been archived by the owner on Dec 18, 2024. It is now read-only.

Making jemalloc an optional feature #468

Merged

Conversation

SebastianSchildt
Copy link
Contributor

Should solve #460

@SebastianSchildt SebastianSchildt force-pushed the fix/making_jemalloc_optional branch from 0cce7f5 to f14237e Compare January 25, 2023 14:44
@SebastianSchildt SebastianSchildt marked this pull request as ready for review January 25, 2023 15:18
Copy link
Contributor

@argerus argerus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, .
But can I suggest that we call the feature jemalloc instead.

kuksa_databroker/databroker/Cargo.toml Outdated Show resolved Hide resolved
kuksa_databroker/databroker/src/main.rs Outdated Show resolved Hide resolved
Signed-off-by: Sebastian Schildt <[email protected]>
@SebastianSchildt SebastianSchildt force-pushed the fix/making_jemalloc_optional branch from f14237e to 60e7a35 Compare January 26, 2023 11:25
@SebastianSchildt
Copy link
Contributor Author

Fixed.
Probably not directly related, but my cargo was thinking serde_derive was missing from Cargo.lock, so I also added it, so at least version is pinned.

@SebastianSchildt SebastianSchildt merged commit 75446da into eclipse:master Jan 27, 2023
@erikbosch erikbosch deleted the fix/making_jemalloc_optional branch September 29, 2023 09:32
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants