[GH-ISSUE #985] Is published version stale? (release of master needed) #584

Closed
opened 2026-03-15 23:18:13 +03:00 by kerem · 14 comments
Owner

Originally created by @chunyingw on GitHub (Jan 8, 2020).
Original GitHub issue: https://github.com/hickory-dns/hickory-dns/issues/985

Describe the bug
I am downloading trust-dns-resolver crate from crates.io which was published 4 days ago. However the loaded crate seem stable, does not contain the PR#979 (merged 6 days ago) nor PR#975 (merged 11 days ago). Is this expected?

Version:
Crate: trust-dns-resolver
Version: 0.18.1

Originally created by @chunyingw on GitHub (Jan 8, 2020). Original GitHub issue: https://github.com/hickory-dns/hickory-dns/issues/985 **Describe the bug** I am downloading trust-dns-resolver crate from crates.io which was published 4 days ago. However the loaded crate seem stable, does not contain the PR#979 (merged 6 days ago) nor PR#975 (merged 11 days ago). Is this expected? **Version:** Crate: trust-dns-resolver Version: 0.18.1
kerem closed this issue 2026-03-15 23:18:18 +03:00
Author
Owner

@bluejekyll commented on GitHub (Jan 8, 2020):

Yes, that was a patch for a specific bug. It was released out of releases/0.18 specifically targetting a server issue. Do you need me to release those changes? I need to see if we need to bump to 0.19 or not...

<!-- gh-comment-id:572292335 --> @bluejekyll commented on GitHub (Jan 8, 2020): Yes, that was a patch for a specific bug. It was released out of `releases/0.18` specifically targetting a server issue. Do you need me to release those changes? I need to see if we need to bump to 0.19 or not...
Author
Owner

@chunyingw commented on GitHub (Jan 8, 2020):

Yes, please. I am working on porting it into Fuchsia officially.

On Wed, Jan 8, 2020, 5:39 PM Benjamin Fry notifications@github.com wrote:

Yes, that was a patch for a specific bug. It was released out of
releases/0.18 specifically targetting a server issue. Do you need me to
release those changes? I need to see if we need to bump to 0.19 or not...


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/bluejekyll/trust-dns/issues/985?email_source=notifications&email_token=ALDCQHGXHNAU366LL5KIW2TQ4ZI2BA5CNFSM4KEODVD2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEIOHZ3Y#issuecomment-572292335,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALDCQHDTLKT4TAWL57SAYRLQ4ZI2BANCNFSM4KEODVDQ
.

<!-- gh-comment-id:572292890 --> @chunyingw commented on GitHub (Jan 8, 2020): Yes, please. I am working on porting it into Fuchsia officially. On Wed, Jan 8, 2020, 5:39 PM Benjamin Fry <notifications@github.com> wrote: > Yes, that was a patch for a specific bug. It was released out of > releases/0.18 specifically targetting a server issue. Do you need me to > release those changes? I need to see if we need to bump to 0.19 or not... > > — > You are receiving this because you authored the thread. > Reply to this email directly, view it on GitHub > <https://github.com/bluejekyll/trust-dns/issues/985?email_source=notifications&email_token=ALDCQHGXHNAU366LL5KIW2TQ4ZI2BA5CNFSM4KEODVD2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEIOHZ3Y#issuecomment-572292335>, > or unsubscribe > <https://github.com/notifications/unsubscribe-auth/ALDCQHDTLKT4TAWL57SAYRLQ4ZI2BANCNFSM4KEODVDQ> > . >
Author
Owner

@chunyingw commented on GitHub (Jan 9, 2020):

I meant stale instead of stable in the title :)

<!-- gh-comment-id:572721021 --> @chunyingw commented on GitHub (Jan 9, 2020): I meant `stale` instead of `stable` in the title :)
Author
Owner

@bluejekyll commented on GitHub (Jan 11, 2020):

@chunyingw I'm reviewing interfaces, it looks like we have some naming that I want to try and clean up.

<!-- gh-comment-id:573314251 --> @bluejekyll commented on GitHub (Jan 11, 2020): @chunyingw I'm reviewing interfaces, it looks like we have some naming that I want to try and clean up.
Author
Owner

@chunyingw commented on GitHub (Jan 11, 2020):

Sure, thanks for cleaning up. I tried the the new version on Fuchsia, it works well.

<!-- gh-comment-id:573334328 --> @chunyingw commented on GitHub (Jan 11, 2020): Sure, thanks for cleaning up. I tried the the new version on Fuchsia, it works well.
Author
Owner

@bluejekyll commented on GitHub (Jan 11, 2020):

FYI. On that PR, I want to drop the usage of failure and convert to thiserror, if it’s not a ton of work. Do you have any issue with that?

<!-- gh-comment-id:573334587 --> @bluejekyll commented on GitHub (Jan 11, 2020): FYI. On that PR, I want to drop the usage of `failure` and convert to `thiserror`, if it’s not a ton of work. Do you have any issue with that?
Author
Owner

@bluejekyll commented on GitHub (Jan 11, 2020):

Now’s a good time to do that since I have to bump to 0.19 anyway.

<!-- gh-comment-id:573334654 --> @bluejekyll commented on GitHub (Jan 11, 2020): Now’s a good time to do that since I have to bump to 0.19 anyway.
Author
Owner

@chunyingw commented on GitHub (Jan 11, 2020):

It is great. No issue for me. We dropped failure on Fuchsia a couple of
weeks ago.

On Sat, Jan 11, 2020, 12:04 PM Benjamin Fry notifications@github.com
wrote:

FYI. On that PR, I want to drop the usage of failure and convert to
thiserror, if it’s not a ton of work. Do you have any issue with that?


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/bluejekyll/trust-dns/issues/985?email_source=notifications&email_token=ALDCQHB4DOM2EVAODGD2QUTQ5H33JA5CNFSM4KEODVD2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEIWGIOY#issuecomment-573334587,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALDCQHE45MR55X3TCL3GQVLQ5H33JANCNFSM4KEODVDQ
.

<!-- gh-comment-id:573335143 --> @chunyingw commented on GitHub (Jan 11, 2020): It is great. No issue for me. We dropped failure on Fuchsia a couple of weeks ago. On Sat, Jan 11, 2020, 12:04 PM Benjamin Fry <notifications@github.com> wrote: > FYI. On that PR, I want to drop the usage of failure and convert to > thiserror, if it’s not a ton of work. Do you have any issue with that? > > — > You are receiving this because you were mentioned. > Reply to this email directly, view it on GitHub > <https://github.com/bluejekyll/trust-dns/issues/985?email_source=notifications&email_token=ALDCQHB4DOM2EVAODGD2QUTQ5H33JA5CNFSM4KEODVD2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEIWGIOY#issuecomment-573334587>, > or unsubscribe > <https://github.com/notifications/unsubscribe-auth/ALDCQHE45MR55X3TCL3GQVLQ5H33JANCNFSM4KEODVDQ> > . >
Author
Owner

@bluejekyll commented on GitHub (Jan 14, 2020):

Got delayed on this over the weekend and yesterday. I'll be pushing a new release later today.

<!-- gh-comment-id:574326049 --> @bluejekyll commented on GitHub (Jan 14, 2020): Got delayed on this over the weekend and yesterday. I'll be pushing a new release later today.
Author
Owner

@chunyingw commented on GitHub (Jan 14, 2020):

Cool, thx.

<!-- gh-comment-id:574339658 --> @chunyingw commented on GitHub (Jan 14, 2020): Cool, thx.
Author
Owner

@bluejekyll commented on GitHub (Jan 15, 2020):

0.19.0 was published: https://crates.io/crates/trust-dns-resolver/0.19.0

<!-- gh-comment-id:574785261 --> @bluejekyll commented on GitHub (Jan 15, 2020): 0.19.0 was published: https://crates.io/crates/trust-dns-resolver/0.19.0
Author
Owner

@chunyingw commented on GitHub (Jan 15, 2020):

Thanks. I tried it out on Fuchsia. trust-dns-proto is good to go.
However because the way trust-dns-resolver Cargo.toml file is written, by default the tokio-runtime feature of trust-dns-proto is enabled, so it is pulling down tokio crates as well, which is not good. I need to solve this issue before I can port trust-dns-resolver to Fuchsia.

<!-- gh-comment-id:574861061 --> @chunyingw commented on GitHub (Jan 15, 2020): Thanks. I tried it out on Fuchsia. trust-dns-proto is good to go. However because the way trust-dns-resolver Cargo.toml file is written, by default the tokio-runtime feature of trust-dns-proto is enabled, so it is pulling down tokio crates as well, which is not good. I need to solve this issue before I can port trust-dns-resolver to Fuchsia.
Author
Owner

@bluejekyll commented on GitHub (Jan 16, 2020):

Ok. I think changeling the dependency from resolver to proto to pull proto with no default features would fix that, right?

<!-- gh-comment-id:574924030 --> @bluejekyll commented on GitHub (Jan 16, 2020): Ok. I think changeling the dependency from resolver to proto to pull proto with no default features would fix that, right?
Author
Owner

@chunyingw commented on GitHub (Jan 16, 2020):

Yes that is what I am thinking as well. I will work on that tomorrow.

On Wed, Jan 15, 2020, 7:37 PM Benjamin Fry notifications@github.com wrote:

Ok. I think changeling the dependency from resolver to proto to pull proto
with no default features would fix that, right?


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/bluejekyll/trust-dns/issues/985?email_source=notifications&email_token=ALDCQHE4X7H5D3Q3ZNARMUDQ56T5LA5CNFSM4KEODVD2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEJCKJ7Q#issuecomment-574924030,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALDCQHCQ466YAY7CNKUYUXLQ56T5LANCNFSM4KEODVDQ
.

<!-- gh-comment-id:574926666 --> @chunyingw commented on GitHub (Jan 16, 2020): Yes that is what I am thinking as well. I will work on that tomorrow. On Wed, Jan 15, 2020, 7:37 PM Benjamin Fry <notifications@github.com> wrote: > Ok. I think changeling the dependency from resolver to proto to pull proto > with no default features would fix that, right? > > — > You are receiving this because you were mentioned. > Reply to this email directly, view it on GitHub > <https://github.com/bluejekyll/trust-dns/issues/985?email_source=notifications&email_token=ALDCQHE4X7H5D3Q3ZNARMUDQ56T5LA5CNFSM4KEODVD2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEJCKJ7Q#issuecomment-574924030>, > or unsubscribe > <https://github.com/notifications/unsubscribe-auth/ALDCQHCQ466YAY7CNKUYUXLQ56T5LANCNFSM4KEODVDQ> > . >
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/hickory-dns#584
No description provided.