Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Look into removing or simplifying xsltproc requirement #104

Closed
gbirchmeier opened this issue Sep 10, 2012 · 4 comments
Closed

Look into removing or simplifying xsltproc requirement #104

gbirchmeier opened this issue Sep 10, 2012 · 4 comments
Milestone

Comments

@gbirchmeier
Copy link
Member

All xsltproc does is create a pretty html-ized output for UTs.

It's kind of annoying to install on Windows because of its dependencies.

I think it'd be good if we can either remove it or make it really easy to install.

One possible option: Use Ruby (with or without a gem) to do xsltproc's job. Since tests are mainly going to be run by people hacking on QF/n, and those developers will already have Ruby, I see no big downside to this (aside from increasing our reliance on Ruby).

@unclepaul84
Copy link
Contributor

I just wrote a .Net based drop-in replacement for xsltproc. it has no dependecies, besides .NET framework. I tested it with the unit test runner script.

See http://unclepaul84.blogspot.com/2012/10/net-equivalent-of-xslproc.html for code.

@gbirchmeier
Copy link
Member Author

That's pretty cool. If I were to use this, would I just drop in the compiled exe?

@unclepaul84
Copy link
Contributor

Yep.

Furthermore, I don't think you would want to assume that it is in the
system PATH in your test script (i would put it in resources and have the
relative path point there), that way installation is much simpler.

2012/10/8 Grant Birchmeier [email protected]

That's pretty cool. If I were to use this, would I just drop in the
compiled exe?


Reply to this email directly or view it on GitHubhttps://github.com//issues/104#issuecomment-9240842.

@cbusbey cbusbey added this to the 2016 Q3 milestone Jul 7, 2016
@cbusbey
Copy link
Contributor

cbusbey commented Jul 7, 2016

  • remove xslt execution calls
  • remove legacy xslt templates
  • remove reference in readme

@cbusbey cbusbey mentioned this issue Aug 12, 2016
@cbusbey cbusbey closed this as completed Aug 12, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants