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

journald: disable default features of tracing-subscriber #1476

Merged
merged 3 commits into from
May 9, 2022
Merged

journald: disable default features of tracing-subscriber #1476

merged 3 commits into from
May 9, 2022

Conversation

themaxdavitt
Copy link
Contributor

Motivation

Closes #1465.

Solution

I'm just disabling the default features of tracing-journald's dependency on tracing-subscriber. The original issue talked about the crate's dependencies more broadly but considering that the standard library is already depended upon I didn't think it made sense to change the tracing-core dependency's features (which are just no_std support).

@themaxdavitt themaxdavitt requested review from davidbarsky, hawkw and a team as code owners July 19, 2021 18:30
@themaxdavitt themaxdavitt changed the title journald: disable default features of tracing-subscriber WIP: journald: disable default features of tracing-subscriber Jul 19, 2021
@themaxdavitt themaxdavitt marked this pull request as draft July 19, 2021 18:46
@themaxdavitt themaxdavitt changed the title WIP: journald: disable default features of tracing-subscriber journald: disable default features of tracing-subscriber Jul 19, 2021
@themaxdavitt themaxdavitt marked this pull request as ready for review July 19, 2021 19:08
@swsnr
Copy link
Contributor

swsnr commented Nov 3, 2021

I noticed that this affects me as well, so I'd be quite interested in seeing this merged and released.

@bryangarza
Copy link
Member

(Going through all open PRs)

This one would be an easy one to rebase and get merged.

@hawkw hawkw enabled auto-merge (squash) May 9, 2022 21:49
@hawkw hawkw merged commit 6bae73b into tokio-rs:master May 9, 2022
hawkw pushed a commit that referenced this pull request Jun 6, 2022
## Motivation

Closes #1465.

## Solution

I'm just disabling the default features of `tracing-journald`'s
dependency on `tracing-subscriber`. The original issue talked about the
crate's dependencies more broadly but considering that the standard
library is already depended upon I didn't think it made sense to change
the `tracing-core` dependency's features (which are just `no_std`
support).
hawkw pushed a commit that referenced this pull request Jun 7, 2022
## Motivation

Closes #1465.

## Solution

I'm just disabling the default features of `tracing-journald`'s
dependency on `tracing-subscriber`. The original issue talked about the
crate's dependencies more broadly but considering that the standard
library is already depended upon I didn't think it made sense to change
the `tracing-core` dependency's features (which are just `no_std`
support).
@themaxdavitt themaxdavitt deleted the journald-less-defaults branch November 6, 2022 03:09
hds added a commit that referenced this pull request Nov 29, 2024
# 0.3.1 (November 29, 2024)

[ [crates.io][crate-0.3.1] ] | [ [docs.rs][docs-0.3.1] ]

### Changed

- disable default features of tracing-subscriber ([#1476])
- allow custom journal fields ([#2708])
- Bump MSRV to 1.63 ([#2793])
- make level mappings configurable ([#2824])

[#1476]: #1476
[#2708]: #2708
[#2793]: #2793
[#2824]: #2824
[docs-0.3.1]: https://docs.rs/tracing-journald/0.3.1
[crate-0.3.1]: https://crates.io/crates/tracing-journald/0.3.1
hds added a commit that referenced this pull request Nov 29, 2024
# 0.3.1 (November 29, 2024)

[ [crates.io][crate-0.3.1] ] | [ [docs.rs][docs-0.3.1] ]

### Changed

- disable default features of tracing-subscriber ([#1476])
- allow custom journal fields ([#2708])
- Bump MSRV to 1.63 ([#2793])
- make level mappings configurable ([#2824])

[#1476]: #1476
[#2708]: #2708
[#2793]: #2793
[#2824]: #2824
[docs-0.3.1]: https://docs.rs/tracing-journald/0.3.1
[crate-0.3.1]: https://crates.io/crates/tracing-journald/0.3.1
hds added a commit that referenced this pull request Nov 29, 2024
# 0.3.1 (November 29, 2024)

[ [crates.io][crate-0.3.1] ] | [ [docs.rs][docs-0.3.1] ]

### Changed

- disable default features of tracing-subscriber ([#1476])
- allow custom journal fields ([#2708])
- Bump MSRV to 1.63 ([#2793])
- make level mappings configurable ([#2824])

[#1476]: #1476
[#2708]: #2708
[#2793]: #2793
[#2824]: #2824
[docs-0.3.1]: https://docs.rs/tracing-journald/0.3.1
[crate-0.3.1]: https://crates.io/crates/tracing-journald/0.3.1
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.

tracing-journald pulling in default features of dependencies
4 participants