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

remove bsdp package #520

Merged
merged 1 commit into from
Dec 6, 2023
Merged

remove bsdp package #520

merged 1 commit into from
Dec 6, 2023

Conversation

pmazzini
Copy link
Collaborator

@pmazzini pmazzini commented Dec 4, 2023

New Mac's no longer support PXE booting. This code is no longer being used with DHCPLB. Fixes #116

Copy link

codecov bot commented Dec 4, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (6a2c8fb) 72.77% compared to head (18c5a71) 73.13%.
Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #520      +/-   ##
==========================================
+ Coverage   72.77%   73.13%   +0.36%     
==========================================
  Files          91       80      -11     
  Lines        5777     5059     -718     
==========================================
- Hits         4204     3700     -504     
+ Misses       1398     1216     -182     
+ Partials      175      143      -32     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@pmazzini pmazzini changed the title remove bsdp code remove bsdp package Dec 4, 2023
@insomniacslk
Copy link
Owner

older macs still support it though. @pmazzini what do we lose leaving this as it is?

@pmazzini
Copy link
Collaborator Author

pmazzini commented Dec 5, 2023

Older Macs have been phased out. I think nobody is making use of this code at the moment.

@insomniacslk
Copy link
Owner

Older Macs have been phased out. I think nobody is making use of this code at the moment.

Phased out as in they are not sold anymore? If so, I still expect that those devices will be around for years to come

@insomniacslk
Copy link
Owner

My point being: I expect that this hardware will lie around in the public market for years still, even if DHCPLB doesn't need then anymore. If there is no downside in removing this code, I'd leave it here for external users

@pmazzini
Copy link
Collaborator Author

pmazzini commented Dec 5, 2023

afaik the hardware is no longer being sold and the software running on it is no longer maintained / supported.

One benefit of cleaning this up is that we will have one less thing to migrate to the new DHCP client. (ref #459 where there was a previous attempt at removing part of this package).

I have't seen users using this code and would rather avoid leaving stale code around. I thought we would benefit from some cleanup but we can leave it if you think somebody may benefit from it.

@insomniacslk
Copy link
Owner

Sounds good, thanks for the details. Go ahead!

@insomniacslk insomniacslk merged commit 8c70d40 into master Dec 6, 2023
12 checks passed
@insomniacslk insomniacslk deleted the bsdp branch December 6, 2023 06:48
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.

Add modifiers to BSDP package
2 participants