Skip to content
Snippets Groups Projects

unify build systems of buffyboard and unl0kr

Merged Imported Administrator requested to merge meson into master
1 unresolved thread

Merge request reports

Merged by AdministratorAdministrator 6 months ago (Sep 26, 2024 5:39pm UTC)

Loading

Pipeline #141385 passed

Pipeline passed for 83509c53 on master

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Author Owner

    Neat! This makes the build files a lot easier and less duplicative. Thank you!

    The only thing I noticed is that libxkbcommon probably shouldn't be a shared dependency.

    By Johannes Marbach on 2024-09-25T19:02:24

  • Administrator requested changes · Imported

    requested changes

    By Johannes Marbach on 2024-09-25T19:02:24

  • Administrator resolved all threads · Imported

    resolved all threads

    By Владимир Стоякин on 2024-09-26T08:02:50

  • Administrator added 1 commit · Imported

    added 1 commit

    • 6f48daca - unify build systems of buffyboard and unl0kr

    Compare with previous version

    By Владимир Стоякин on 2024-09-26T08:11:42

    • Author Owner

      Side notes.

      1. Is it okay?
      $./find-lvgl-sources.sh lvgl > /dev/null
      find: 'lvgl/src/dev': No such file or directory
      find: 'lvgl/src/extra': No such file or directory
      find: 'lvgl/src/extra/widgets/keyboard': No such file or directory
      find: 'lvgl/src/extra/widgets/msgbox': No such file or directory
      find: 'lvgl/src/extra/widgets/span': No such file or directory
      find: 'lvgl/src/hal': No such file or directory
      1. Why do you use sort for machine input?
      find "$1/src" -name 'lv_init.c' | LC_ALL=C sort

      By Владимир Стоякин on 2024-09-26T17:37:12

      Edited by Ghost User
    • Author Owner

      Re 1: Oh, looks like those folders don't exist anymore. This doesn't (appear to) break anything but we should probably remove these folders from the script. I'll just do that on master separately. Thanks for flagging this.

      Re 2: This came from !24 (diffs). It appears to matter for strictly reproducible builds.

      By Johannes Marbach on 2024-09-26T17:37:12

    • Please register or sign in to reply
  • Administrator added 1 commit · Imported

    added 1 commit

    • ec21e5ea - unify build systems of buffyboard and unl0kr

    Compare with previous version

    By Владимир Стоякин on 2024-09-26T10:46:29

  • Author Owner

    Looks great, thanks you very much! I'll add you to the changelog on master after merging.

    By Johannes Marbach on 2024-09-26T17:39:17

  • Administrator approved this merge request · Imported

    approved this merge request

    By Johannes Marbach on 2024-09-26T17:39:24

  • Administrator mentioned in commit 83509c53 · Imported

    mentioned in commit 83509c53

    By Johannes Marbach on 2024-09-26T17:39:36

  • Administrator merged · Imported

    merged

  • Please register or sign in to reply
    Loading