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

chore: prepare tracing-journald 0.4.0 #2791

Open
wants to merge 14 commits into
base: v0.1.x
Choose a base branch
from

Conversation

davidbarsky
Copy link
Member

Stacked atop of #2790.

@davidbarsky davidbarsky requested review from a team and hawkw as code owners November 6, 2023 22:04
@davidbarsky davidbarsky force-pushed the david/prepare-tracing-journald-0.4.0 branch from 194dd34 to 146e5d8 Compare November 12, 2023 22:10
@Finomnis
Copy link
Contributor

Hey! Any news?

@hongquan
Copy link

Please include #2824

@hawkw
Copy link
Member

hawkw commented Jan 11, 2024

I've merged #2824, so we should be able to cherry-pick that as well.

@Finomnis
Copy link
Contributor

Finomnis commented Mar 11, 2024

@hawkw Any idea when this will actually be released? This is stale for quite a while now ... and I'd be strongly interested in it for a work project. Specifically the custom journal fields.

@Finomnis
Copy link
Contributor

@hawkw Is there anything I can do to get this rolling again? Or did it get lost somewhere?

@Finomnis
Copy link
Contributor

@davidbarsky What's the showstopper here?

@hongquan
Copy link

Please make progress.

@Finomnis
Copy link
Contributor

Finomnis commented Jun 10, 2024

Still nothing? What's the showstopper, is the backport complexity bigger than expected? Are code changes required, or open issues? I am willed to assist if I can.

@jmagnuson
Copy link

I don't know if the CI warnings are what's holding this back, but both current issues would be addressed by merging in #2961, I think.

Also worth noting that #2958 also proposes a fix to the shared reference to mutable static issue, but carries a MSRV bump with it.

@davidbarsky
Copy link
Member Author

It's the CI warning that are holding it back, which is pretty annoying.

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.

5 participants