hi, a second try to make a common scripts. i hope it'll work. i still not enable mingw32.sh, but it also would be useful with this patch.
thanks for adding mingw32-make, alot of programs need's it to build
On 11/25/2008 3:34 PM, Farkas Levente wrote:
hi, a second try to make a common scripts. i hope it'll work. i still not enable mingw32.sh, but it also would be useful with this patch.
fedora-mingw mailing list fedora-mingw@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/fedora-mingw
the best would be to use %_mingw32_make. mingw32-make just to use during development.
Itamar - IspBrasil wrote:
thanks for adding mingw32-make, alot of programs need's it to build
On 11/25/2008 3:34 PM, Farkas Levente wrote:
hi, a second try to make a common scripts. i hope it'll work. i still not enable mingw32.sh, but it also would be useful with this patch.
fedora-mingw mailing list fedora-mingw@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/fedora-mingw
On Tue, Nov 25, 2008 at 06:34:20PM +0100, Farkas Levente wrote:
hi, a second try to make a common scripts. i hope it'll work. i still not enable mingw32.sh, but it also would be useful with this patch.
I don't have time now, but I will check this tomorrow.
Have you tried to use strace on mingw32-make? Last time I saw that the %{name} and %{version} macros were not being substituted into the variables on the make command line, which is why I rejected that patch.
Rich.
Richard W.M. Jones wrote:
On Tue, Nov 25, 2008 at 06:34:20PM +0100, Farkas Levente wrote:
hi, a second try to make a common scripts. i hope it'll work. i still not enable mingw32.sh, but it also would be useful with this patch.
I don't have time now, but I will check this tomorrow.
Have you tried to use strace on mingw32-make? Last time I saw that the %{name} and %{version} macros were not being substituted into the variables on the make command line, which is why I rejected that patch.
i don't really understand it. i try this both from spec file and command line, but there is no %{name} and %{version} in it? what can i looking for in a strace output?
On Tue, Nov 25, 2008 at 11:30:49PM +0100, Farkas Levente wrote:
Richard W.M. Jones wrote:
On Tue, Nov 25, 2008 at 06:34:20PM +0100, Farkas Levente wrote:
hi, a second try to make a common scripts. i hope it'll work. i still not enable mingw32.sh, but it also would be useful with this patch.
I don't have time now, but I will check this tomorrow.
Have you tried to use strace on mingw32-make? Last time I saw that the %{name} and %{version} macros were not being substituted into the variables on the make command line, which is why I rejected that patch.
i don't really understand it. i try this both from spec file and command line, but there is no %{name} and %{version} in it? what can i looking for in a strace output?
OK, that's encouraging, but I'll check for sure tomorrow. You need to do something like:
strace -f -o /tmp/log -s 4096 mingw32-make grep '%' /tmp/log | less
Rich.
Richard W.M. Jones wrote:
On Tue, Nov 25, 2008 at 11:30:49PM +0100, Farkas Levente wrote:
Richard W.M. Jones wrote:
On Tue, Nov 25, 2008 at 06:34:20PM +0100, Farkas Levente wrote:
hi, a second try to make a common scripts. i hope it'll work. i still not enable mingw32.sh, but it also would be useful with this patch.
I don't have time now, but I will check this tomorrow.
Have you tried to use strace on mingw32-make? Last time I saw that the %{name} and %{version} macros were not being substituted into the variables on the make command line, which is why I rejected that patch.
i don't really understand it. i try this both from spec file and command line, but there is no %{name} and %{version} in it? what can i looking for in a strace output?
OK, that's encouraging, but I'll check for sure tomorrow. You need to do something like:
strace -f -o /tmp/log -s 4096 mingw32-make grep '%' /tmp/log | less
i can't see anything interesting in it. anyway here a modified (two char) version of the previous patch (which looks better).
On Wed, Nov 26, 2008 at 10:33:38AM +0100, Farkas Levente wrote:
+NAME="_`basename $0|tr -- - _`" +eval "`rpm --eval "%{$NAME}"`" "$@"
The quoting here is confusing. How about using $(..) instead of `..` ?
Index: mingw32.sh
We need mingw32.csh too. Nobody uses CSH, but it's tradition to provide both Bourne shell and C-shell compatible scripts.
Rich.
Richard W.M. Jones wrote:
On Wed, Nov 26, 2008 at 10:33:38AM +0100, Farkas Levente wrote:
+NAME="_`basename $0|tr -- - _`" +eval "`rpm --eval "%{$NAME}"`" "$@"
The quoting here is confusing. How about using $(..) instead of `..` ?
$() is bash-izm:-( inside `..` it's a subsell so " is protected by the subsell (ie. no need for ", around $@ the " is highly is always preferred. so imho it's the best syntax.
Index: mingw32.sh
We need mingw32.csh too. Nobody uses CSH, but it's tradition to provide both Bourne shell and C-shell compatible scripts.
the main problem here is that spec is bash like ie. %mingw32_env is only good for bash and all other macro is good for only bash:-( ie. there is no easy way to auto generate .csh that's why i leave it empty. but it'd be useful to give a change to someone easily set the current shell's env (and i can't any other way to do so just using alias).