Setup winxed #19

Merged
merged 6 commits into from Nov 4, 2011

3 participants

@lateau

I'm so late, because I had a high fever.

setup.winxed allows almost targets of setup.pir.
'smoke' and 'test' works but not fully, because seems like WINXED does not allows named parmeters in pir.
so 'runtests' cannot be called and it causes 'no test'.
I'm not sure it's bug but I'll report it to NotFound then modify this problems.

And some codes are very dirty but they will be fixed.

$ winxed --version
1.2.0
$ winxed setup.winxed test
.......
.......

Files=28, Tests=0, 0.172 wallclock secs <<<<<< HERE, "Tests=0"
Result: FAIL
creat report.tar.gz

@NotFound
Parrot Virtual Machine member

Winxed pirops are not, and will never be, a generic inlining pir feature. They don't expect pir sintactic sugar, just take an op and its operands, and the operands are winxed expressions.

All parrot calling conventions are supported via modifiers in parameters and arguments. No pir required.

@leto
Parrot Virtual Machine member

@NotFound: can you help refactor this so the smoke and test targets work? I don't have the tuits.

@NotFound
Parrot Virtual Machine member

Fixed and added in the branch winxed_setup, 3f3b4a7

@lateau lateau setup.winxed
this is a replacement of setup.pir
all targets of setup.pir are supported and works now:
  * build
  * test
  * install
  * uninstall
  * clean
  * update
  * plumage
  * sdist
  * bdist
  * smoke
  * help
changed:
  * clean: remove rpm directory and it's subdirectories also
  * build: build target will be runed before sdist and bdist
Thanks NotFound++
32929b6
@lateau

setup.winxed is tested with winxed1.2.0 and works now.
I found a way from comments of NotFound to me.
I tested the builder on fedora15 amd64 and osx10.7 only, so more tests are needed on some other machine.

many thanks:
@leto
@NotFound

@leto
Parrot Virtual Machine member

This looks good. Is there a reason the POD is commented out?

@lateau

Because I got a error message with pod in winxed "setup.winxed:3: Unexpected token near =".
And I cannot found any informations of pod with winxed at these pages:

http://winxed.org/
http://whiteknight.github.com/Rosella/winxed/index.html

Anyway, If we can use pod for some informations in winxed, that's much pretty better than some others.

@leto
Parrot Virtual Machine member
@lateau

much better!

@leto++

@leto leto closed this Nov 3, 2011
@lateau

not merged. is NotFound doing this?

Sent via Hubroid

@leto leto reopened this Nov 4, 2011
@leto leto merged commit 97588e0 into parrot:master Nov 4, 2011
@leto
Parrot Virtual Machine member

Accidentally closed this before, without pressing the shiny merge button first.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment