Skip to content

Conversation

@yadij
Copy link
Contributor

@yadij yadij commented Jan 7, 2026

Make sure the stub provides all the symbols declared in fd.h

@yadij yadij added M-cleared-for-merge https://github.com/measurement-factory/anubis#pull-request-labels S-could-use-an-approval An approval may speed this PR merger (but is not required) backport-to-v7 maintainer has approved these changes for v7 backporting labels Jan 7, 2026
@yadij yadij changed the title Update stub_fd.cc Maintenance: Update stub_fd.cc Jan 7, 2026
@yadij
Copy link
Contributor Author

yadij commented Jan 7, 2026

NB: maintenance because this is not a bug fix, just removing technical debt found while refactoring other code.

@rousskov rousskov changed the title Maintenance: Update stub_fd.cc Maintenance: Add missing definitions to stub_fd.cc Jan 7, 2026
Copy link
Contributor

@rousskov rousskov left a comment

Choose a reason for hiding this comment

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

I adjusted PR title to be more specific.

@rousskov rousskov added S-waiting-for-author author action is expected (and usually required) and removed M-cleared-for-merge https://github.com/measurement-factory/anubis#pull-request-labels S-could-use-an-approval An approval may speed this PR merger (but is not required) labels Jan 7, 2026
@yadij yadij added the S-waiting-for-reviewer ready for review: Set this when requesting a (re)review using GitHub PR Reviewers box label Jan 8, 2026
@rousskov rousskov removed the S-waiting-for-reviewer ready for review: Set this when requesting a (re)review using GitHub PR Reviewers box label Jan 8, 2026
@rousskov rousskov dismissed their stale review January 8, 2026 21:06

I should not insist on improving that moved stub code in this PR.

squid-anubis pushed a commit that referenced this pull request Jan 17, 2026
Make sure the stub provides all the symbols declared in fd.h
@squid-anubis squid-anubis added M-waiting-staging-checks https://github.com/measurement-factory/anubis#pull-request-labels M-passed-staging-checks https://github.com/measurement-factory/anubis#pull-request-labels and removed M-waiting-staging-checks https://github.com/measurement-factory/anubis#pull-request-labels labels Jan 17, 2026
@yadij yadij added M-cleared-for-merge https://github.com/measurement-factory/anubis#pull-request-labels and removed S-waiting-for-author author action is expected (and usually required) M-passed-staging-checks https://github.com/measurement-factory/anubis#pull-request-labels labels Jan 17, 2026
@squid-anubis squid-anubis added M-merged https://github.com/measurement-factory/anubis#pull-request-labels and removed M-cleared-for-merge https://github.com/measurement-factory/anubis#pull-request-labels labels Jan 17, 2026
@yadij yadij deleted the arc-stub_fd-1 branch January 17, 2026 22:14
squidadm pushed a commit to squidadm/squid that referenced this pull request Jan 18, 2026
Make sure the stub provides all the symbols declared in fd.h
@squidadm
Copy link
Collaborator

queued for backport to v7

@squidadm squidadm removed the backport-to-v7 maintainer has approved these changes for v7 backporting label Jan 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

M-merged https://github.com/measurement-factory/anubis#pull-request-labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants