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

simplify ptpheck map #406

Closed
wants to merge 2 commits into from
Closed

simplify ptpheck map #406

wants to merge 2 commits into from

Conversation

yarikk
Copy link
Contributor

@yarikk yarikk commented Oct 15, 2024

Summary:
Simplify implementation of the ptpcheck map command.

Deprecate phc.IfacesInfo which is a mere combination of net.Interfaces and phc.IfaceInfo, was only used in ptpcheck map which is simpler AND more optimal without it.

Differential Revision: D64395255

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Oct 15, 2024
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D64395255

yarikk added a commit to yarikk/time that referenced this pull request Oct 15, 2024
Summary:

Simplify implementation of the `ptpcheck map` command.

Deprecate `phc.IfacesInfo` which is a mere combination of `net.Interfaces` and `phc.IfaceInfo`, was only used in `ptpcheck map` which is simpler AND more optimal without it.

Differential Revision: D64395255
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D64395255

yarikk added a commit to yarikk/time that referenced this pull request Oct 15, 2024
Summary:

Simplify implementation of the `ptpcheck map` command.

Deprecate `phc.IfacesInfo` which is a mere combination of `net.Interfaces` and `phc.IfaceInfo`, was only used in `ptpcheck map` which is simpler AND more optimal without it.

Reviewed By: t3lurid3

Differential Revision: D64395255
yarikk added a commit to yarikk/time that referenced this pull request Oct 15, 2024
Summary:

Simplify implementation of the `ptpcheck map` command.

Deprecate `phc.IfacesInfo` which is a mere combination of `net.Interfaces` and `phc.IfaceInfo`, was only used in `ptpcheck map` which is simpler AND more optimal without it.

Reviewed By: t3lurid3

Differential Revision: D64395255
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D64395255

Summary:

Deprecate `DeviceFromIface` function which duplicates the functionality of `IfaceToPHCDevice` – retrieving a path to `/dev/ptp𝑛` corresponding to a given network interface. The later is already optimal and has more identifiable references.

Reviewed By: abulimov

Differential Revision: D64332922
Summary:

Simplify implementation of the `ptpcheck map` command.

Deprecate `phc.IfacesInfo` which is a mere combination of `net.Interfaces` and `phc.IfaceInfo`, was only used in `ptpcheck map` which is simpler AND more optimal without it.

Reviewed By: t3lurid3

Differential Revision: D64395255
yarikk added a commit to yarikk/time that referenced this pull request Oct 15, 2024
Summary:

Simplify implementation of the `ptpcheck map` command.

Deprecate `phc.IfacesInfo` which is a mere combination of `net.Interfaces` and `phc.IfaceInfo`, was only used in `ptpcheck map` which is simpler AND more optimal without it.

Reviewed By: t3lurid3

Differential Revision: D64395255
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D64395255

@facebook-github-bot
Copy link
Contributor

This pull request has been merged in 5e20cc7.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported Merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants