Skip to content

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:

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 the CHANGELOG.md.
Edited by 🤖 GitLab Bot 🤖

Merge request reports

Loading