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 build scripts & b1's cicd #137

Merged
merged 3 commits into from
Apr 28, 2022
Merged

remove build scripts & b1's cicd #137

merged 3 commits into from
Apr 28, 2022

Conversation

spoonincode
Copy link
Member

Remove the entirety of build scripts & b1's CICD. Both of these are untested in Mandel's repo, are rotting away (already incorrect in places), and are unmaintained (none of the Mandel developers use the scripts).

There is no immediate replacement for any of the removed functionality beyond what is already present. Probably the most significant loss is the pinned build script as that was still somewhat arguably user facing and offered some utility that has no current substitute. The desire is to have a reasonable replacement before 3.1rc1 is ready.

I see no real downside with removing all this now, with the slight annoyance of a break in git history for anything restored later (something like the package creation scripts could see a revival in some form or another).

Copy link
Member

@heifner heifner left a comment

Choose a reason for hiding this comment

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

I find the pinned scripts useful to follow when setting up a dev environment. Can we hold off on this until the "reasonable replacement before 3.1rc1 is ready." ?

@spoonincode
Copy link
Member Author

The apt-get commands in the README.md work as far as I know (and that command is being kept up to date; e.g. removal of libgmp-dev in recent PR).

Which part are you finding useful? It may be better for me to just move it to README

@heifner
Copy link
Member

heifner commented Apr 26, 2022

Which part are you finding useful? It may be better for me to just move it to README

If you are a node operator and you don't want your state file to fail to load, you want to make sure you do a pinned build. Otherwise dependencies like boost could change on you. The pinned builds document what is used to build the releases. Of course, we are currently not building releases. But we should document and recommend a "pinned" stable env for those who build Mandel and want to be sure a restart of nodeos without a snapshot works.

The entire purpose of this test was to check that the build was
performed with b1's cicd expected boost version. Without b1's cicd
it's a complete noop test. So remove it
@spoonincode
Copy link
Member Author

I guess the way I see it is that 3.0/3.0.5 have already officially removed that feature. Yeah, it's a super important feature, but by ignoring it, not wiring it up for testing, and then cutting multiple RC releases without it, the project has implicitly officially removed the feature. It's gone.

Keeping the files around doesn't bring back the feature. Instead, leaving rubbish around seems like a disaster waiting to happen; it sets a trap.

If we feel the feature is important, we can add it back in a supported way. Would an issue as simple as 'bring back pinned builds & binaries' work?

@heifner
Copy link
Member

heifner commented Apr 27, 2022

Would an issue as simple as 'bring back pinned builds & binaries' work?

You make a good point. Yes, please add an issue and link it to this.

@spoonincode spoonincode merged commit 3641155 into main Apr 28, 2022
@spoonincode spoonincode deleted the remove_b1_cicd branch April 28, 2022 03: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.

None yet

2 participants