! Please note that this is a snapshot of our old Bugzilla server, which is read only since May 29, 2020. Please go to gitlab.xfce.org for our new server !
make distcheck is broken
Status:
RESOLVED: FIXED
Product:
Xfce4-panel
Component:
General

Comments

Description Skunnyk editbugs 2017-11-27 14:18:06 CET
make distcheck is broken on xfce4-panel when "vala" is present on the system due to the introduction of vala binding generation in commits 3d6d1dd39864d013e96072ce63e8e0c4dfff9a2d and following.

By reading https://cgit.freedesktop.org/accountsservice/commit/?id=17af4612e877c3a5c6bff67064d648f0eb2ea0cc , https://wiki.gnome.org/Projects/Vala/FAQ#Is_having_a_local_copy_of_a_.vapi_in_my_project_good_practice.3F and https://wiki.gnome.org/Projects/Vala/UpstreamGuide#Autotools_Integration , we don't need to provide the vapi in release tarballs.


So I propose to add --enable-vala=no in DISTCHECK_CONFIGURE_FLAGS .
Patch incoming (with other small fixes for distcheck)

The other solution is to remove the vala thing from the code.
Comment 1 Skunnyk editbugs 2017-11-27 22:52:16 CET
Created attachment 7456 
Fix make distcheck
Comment 2 Simon Steinbeiss editbugs 2017-11-30 00:45:16 CET
While fixing typos in your commit message I also managed to sneak one in... :(

https://git.xfce.org/xfce/xfce4-panel/commit/?id=628624b41908eb70264b4ea491394834af5783ef

Thanks for the patch!

Bug #14040

Reported by:
Skunnyk
Reported on: 2017-11-27
Last modified on: 2017-11-30

People

Assignee:
Simon Steinbeiss
CC List:
0 users

Version

Version:
Unspecified

Attachments

Fix make distcheck (829 bytes, patch)
2017-11-27 22:52 CET , Skunnyk
no flags

Additional information