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

test suite cannot be ran without installing first #5710

Closed
vicuna opened this issue Jul 31, 2012 · 5 comments
Closed

test suite cannot be ran without installing first #5710

vicuna opened this issue Jul 31, 2012 · 5 comments
Assignees
Labels
Milestone

Comments

@vicuna
Copy link

vicuna commented Jul 31, 2012

Original bug ID: 5710
Reporter: didc
Assigned to: @damiendoligez
Status: closed (set by @damiendoligez on 2013-05-17T19:32:27Z)
Resolution: fixed
Priority: low
Severity: minor
Version: 4.00.0
Target version: 4.01.0+dev
Fixed in version: 4.01.0+dev
Category: ~DO NOT USE (was: OCaml general)
Has duplicate: #5483
Monitored by: mehdi @glondu

Bug description

The test scripts partly expect the compiler to be installed in order to complete successfully. In certain circumstances, the person compiling the software suite may not have the necessary priviledges, or simply may not wish to do things in that order.

Additional information

I provide a patch which make the necessary changes in the testsuitEs scripts to only rely on the project binaries.
There are some remaining issues though:
1- non unix platforms (ie MS-Windows), are not handled,
2- The scripts use some gnu make specificities (see otherlibs.mk)
3- I tried to handle support for mono 2.0 in the csharp tests and it seems to work, but it may have broken the test for the real tying,
4- I'm not sure that lib threads/systhreads are handled correctly, although everything pass.

Thank you!

File attachments

@vicuna
Copy link
Author

vicuna commented Jul 31, 2012

Comment author: didc

please disregard the first patch, as it contains superfluous files. I provide hereafter an updated patch without these.

@vicuna
Copy link
Author

vicuna commented Aug 1, 2012

Comment author: @glondu

Duplicate of #5483

@vicuna
Copy link
Author

vicuna commented May 17, 2013

Comment author: @damiendoligez

I have applied the Mono part of the patch, and I'm working on the main topic: making the testsuite work without installing, as well as making it work on Windows.

@vicuna
Copy link
Author

vicuna commented May 17, 2013

Comment author: @glondu

Thanks! It will be greatly appreciated.

@vicuna
Copy link
Author

vicuna commented May 17, 2013

Comment author: @damiendoligez

fixed in trunk (r13687)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants