Skip to content
This repository has been archived by the owner on Jul 8, 2022. It is now read-only.

Fix unit tests & build w/o ZeroEQ #30

Merged
merged 2 commits into from
Jun 2, 2015
Merged

Fix unit tests & build w/o ZeroEQ #30

merged 2 commits into from
Jun 2, 2015

Conversation

tribal-tec
Copy link
Contributor

No description provided.

@bbpbuildbot
Copy link

Refer to this link for build results (access rights to CI server needed):
https://bbpcode.epfl.ch/ci//job/oss.Livre.github/35/
Test FAILed.

@bilgili
Copy link
Contributor

bilgili commented Jun 1, 2015

I am checking the zeq issue

@bbpbuildbot
Copy link

Refer to this link for build results (access rights to CI server needed):
https://bbpcode.epfl.ch/ci//job/oss.Livre.github/36/
Test FAILed.

@bbpbuildbot
Copy link

Refer to this link for build results (access rights to CI server needed):
https://bbpcode.epfl.ch/ci//job/oss.Livre.github/39/
Test FAILed.

@tribal-tec tribal-tec changed the title Fix build w/o ZeroEQ Fix unit tests & build w/o ZeroEQ Jun 2, 2015
@tribal-tec
Copy link
Contributor Author

retest this please

@bbpbuildbot
Copy link

Refer to this link for build results (access rights to CI server needed):
https://bbpcode.epfl.ch/ci//job/oss.Livre.github/38/
Test PASSed.

@bbpbuildbot
Copy link

Refer to this link for build results (access rights to CI server needed):
https://bbpcode.epfl.ch/ci//job/oss.Livre.github/40/
Test PASSed.

@tribal-tec
Copy link
Contributor Author

updated. ready to merge?

@@ -75,7 +75,7 @@ void _testDataSource( const std::string& uriStr )
BOOST_AUTO_TEST_CASE( memoryDataSource )
{
std::stringstream volumeName;
volumeName << "mem:///#" << VOXEL_SIZE_X << "," << VOXEL_SIZE_Y << ","
volumeName << "mem://#" << VOXEL_SIZE_X << "," << VOXEL_SIZE_Y << ","
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"men:///#" is valid , but even "mem:#whatever" is a valid URI. The // separator is needed when you are going to specify an authority, to distinguish it from an absolute path (a single /).

@hernando
Copy link
Contributor

hernando commented Jun 2, 2015

+1 if you don't want to use HBPVIS/Servus#10.

bilgili pushed a commit that referenced this pull request Jun 2, 2015
Fix unit tests & build w/o ZeroEQ
@bilgili bilgili merged commit 2f42546 into BlueBrain:master Jun 2, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants