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 BOARD_FYSETC_SPIDER_KING407 #26459

Conversation

sjasonsmith
Copy link
Contributor

Description

When BOARD_FYSETC_SPIDER_KING407 was added (50f7982) it did not include a pins file.
With no pins file in over a year, it is highly unlikely this is being used by anybody.

Benefits

Less code to maintain.

@sjasonsmith
Copy link
Contributor Author

If @GerogeFu is still around, perhaps he can provide information about why this was never used.

@ellensp
Copy link
Contributor

ellensp commented Nov 25, 2023

@sjasonsmith
Copy link
Contributor Author

The missing file is here https://github.com/FYSETC/FYSETC-SPIDER-KING/blob/main/firmware/Marlin/Marlin/Marlin/src/pins/stm32f4/pins_FYSETC_SPIDER_KING407.h

So, do we proactively pull that in, or treat it as an impartial addition and remove it? Does the board actually exist? Does anybody care that it is missing if it hasn't been added in over a year?

@ellensp
Copy link
Contributor

ellensp commented Nov 25, 2023

Well the board is available on aliexpress, so eventually someone will want to put updated marlin on it.
So from my point of view we should pull in the missing pins file, verify it will at least builds, and leave it be.

@sjasonsmith
Copy link
Contributor Author

Well the board is available on aliexpress, so eventually someone will want to put updated marlin on it. So from my point of view we should pull in the missing pins file, verify it will at least build, and leave it be.

@ellensp do you want to do that? I just ran across this because it interfered with me executing a script to run through every board in boards.ini to test symbol changes. It looked unused from the perspective of the Marlin codebase and I didn't go trying to hunt it down.

@ellensp
Copy link
Contributor

ellensp commented Nov 25, 2023

@sjasonsmith yes I can do that. working on it now.

@sjasonsmith
Copy link
Contributor Author

OK, I'll go ahead and close this. Thanks!

@sjasonsmith sjasonsmith deleted the PR/remove_spider_king branch April 21, 2024 16:44
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.

2 participants