Skip to content
Snippets Groups Projects

Add suport to Meson

Merged Andre Miranda requested to merge meson into master
1 unresolved thread

When comparing meson and autotools installed files as instructed here, there is a diff them, but I think meson is correct and I'm not willing to fix autotools:

-./usr/lib/x86_64-pc-linux-gnu
-./usr/lib/x86_64-pc-linux-gnu/xfce4
-./usr/lib/x86_64-pc-linux-gnu/xfce4/panel
-./usr/lib/x86_64-pc-linux-gnu/xfce4/panel/plugins
-./usr/lib/x86_64-pc-linux-gnu/xfce4/panel/plugins/libxfce4dict.so
+./usr/lib/xfce4
+./usr/lib/xfce4/panel
+./usr/lib/xfce4/panel/plugins
+./usr/lib/xfce4/panel/plugins/libxfce4dict.so
Edited by Andre Miranda

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Gaël Bonithon
  • You're also missing a check for locale.h:

    [xfce4-dict]$ grep -Prn --include='*.c*' -- '#ifdef HAVE_(?!CONFIG_H|XFCE_REVISION_H)'
    src/xfce4-dict.c:33:#ifdef HAVE_LOCALE_H
    • When comparing meson and autotools installed files as instructed here, there is a diff them, but I think meson is correct and I'm not willing to fix autotools:

      It's because of a lib prefix that was given for this install, isn't it? It's on debian?

      Edited by Gaël Bonithon
    • Yes, ./autogen.sh --prefix=/usr --libdir=/usr/lib/$(gcc -dumpmachine) from the meson project template, but I didn't see this difference with screenshooter which also has a panel plugin :shrug:

    • Please register or sign in to reply
  • Andre Miranda added 1 commit

    added 1 commit

    Compare with previous version

  • Andre Miranda added 1 commit

    added 1 commit

    • bbdcc027 - Apply 2 suggestion(s) to 2 file(s)

    Compare with previous version

  • Andre Miranda resolved all threads

    resolved all threads

  • Andre Miranda added 1 commit

    added 1 commit

    • 4cf133a2 - Add xfce-revision.h.in to EXTRA_DIST

    Compare with previous version

  • Andre Miranda requested review from @Tamaranch

    requested review from @Tamaranch

  • Gaël Bonithon approved this merge request

    approved this merge request

  • merged

  • Please register or sign in to reply
    Loading