! 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 !
xdt-version script
Status:
RESOLVED: WONTFIX
Product:
Xfce4-dev-tools
Component:
General

Comments

Description Mike Massonnet editbugs 2010-11-11 16:45:16 CET
Created attachment 3183 
Patch

Hi,

Recent packages who updated to xdt-autogen with support for $XDT_AUTOGEN_REQUIRED_VERSION are not working with xfce4-dev-tools < 4.7.

A simple solution (preferably before 4.8 final) is to include a new shell script that the autogen.sh files can check in order to prevent the execution of xdt-autogen.

I had like to suggest the command xdt-version that simply prints a text message. While this command is not useful in reality it is useful for the purpose mentioned above.

See attached patch.

Cheers
Mike
Comment 1 Nick Schermer editbugs 2010-11-12 12:14:05 CET
Maybe it's better to return True/False if an argument is supplied, ie. $(xdt-version "4.7.3"), if no version is set, then echo the version. In that case the logic from xdt-autogen can be moved into xdt-version and call xdt-version from xdt-autogen.sh if XDT_AUTOGEN_REQUIRED_VERSION is set.

However I have no idea how useful this is, xdt 4.7.3 is a build dependency and distros will know this. Apart from that it's only needed to build new packages, releases are not affected by the xdt version.
Comment 2 Mike Massonnet editbugs 2010-11-12 18:06:41 CET
Indeed, but it's not necessary to introduce changes from how it works now, it is mainly to avoid users installing xfce4-dev-tools from their distros and trying to build packages from git.

The goal is to prevent running xdt-autogen starting from the autogen.sh script.
Comment 3 Mike Massonnet editbugs 2011-10-19 09:21:21 CEST
Not sure myself what I wanted here, so setting as WONTFIX :)

Bug #6811

Reported by:
Mike Massonnet
Reported on: 2010-11-11
Last modified on: 2011-10-19

People

Assignee:
Xfce Bug Triage
CC List:
1 user

Version

Attachments

Patch (1.83 KB, patch)
2010-11-11 16:45 CET , Mike Massonnet
no flags

Additional information