Simplify `rake` check in Makefile
-
Please check this box if this contribution uses AI-generated content (including content generated by GitLab Duo features) as outlined in the GitLab DCO & CLA
What does this merge request do and why?
Refactoring rake
+ make
code.
rake
needs zeitwerk
installed, because it imports lib/gdk.rb
, and that means Makefile
need to be modified to ensure zeitwerk
is installed prior to running rake
.
As an alternative, generate Makefile
config with pure Ruby. This will bring bootstrap sequence from:
graph LR
bash --> ruby --> bundle --> make --> rake --> make
To:
graph LR
bash --> ruby --> bundle --> make
How to set up and validate locally
Numbered steps to set up and validate the change are strongly suggested.
Impacted categories
The following categories relate to this merge request:
-
gdk-reliability - e.g. When a GDK action fails to complete. -
gdk-usability - e.g. Improvements or suggestions around how the GDK functions. -
gdk-performance - e.g. When a GDK action is slow or times out.
Merge request checklist
-
This change is backward compatible. If not, please include steps to communicate to our users. -
Tests added for new functionality. If not, please raise an issue to follow-up. -
Documentation added/updated, if needed. -
Announcement added, if change is notable. -
gdk doctor
test added, if needed. -
Add the ~highlight
label if this MR should be included in theCHANGELOG.md
.
Edited by 🤖 GitLab Bot 🤖