Carl Trieloff <[EMAIL PROTECTED]> wrote:
> Doing this patch
Thank you.
> but having build issues. will figure them out tomorrow.
Details? Without those, it's much harder to help.
Here's one more piece, without which "make check" won't compile:
2006-12-06 Jim Meyering <[EMAIL PROTECTED]>
* tests/ConfigurationTest.cpp: Update conf.parse callers.
Index: tests/ConfigurationTest.cpp
===================================================================
--- tests/ConfigurationTest.cpp (revision 482944)
+++ tests/ConfigurationTest.cpp (working copy)
@@ -41,7 +41,7 @@
{
Configuration conf;
char* argv[] = {"ignore", "--help"};
- conf.parse(2, argv);
+ conf.parse("t1", 2, argv);
CPPUNIT_ASSERT(conf.isHelp());
}
@@ -49,7 +49,7 @@
{
Configuration conf;
char* argv[] = {"ignore", "--port", "6789"};
- conf.parse(3, argv);
+ conf.parse("t1", 3, argv);
CPPUNIT_ASSERT_EQUAL(6789, conf.getPort());
}
@@ -57,7 +57,7 @@
{
Configuration conf;
char* argv[] = {"ignore", "-p", "6789"};
- conf.parse(3, argv);
+ conf.parse("t1", 3, argv);
CPPUNIT_ASSERT_EQUAL(6789, conf.getPort());
}
@@ -65,7 +65,7 @@
{
Configuration conf;
char* argv[] = {"ignore", "--store", "my-store-module.so"};
- conf.parse(3, argv);
+ conf.parse("t1", 3, argv);
std::string expected("my-store-module.so");
CPPUNIT_ASSERT_EQUAL(expected, conf.getStore());
}
@@ -74,7 +74,7 @@
{
Configuration conf;
char* argv[] = {"ignore", "-t", "--worker-threads", "10"};
- conf.parse(4, argv);
+ conf.parse("t1", 4, argv);
CPPUNIT_ASSERT_EQUAL(5672, conf.getPort());//default
CPPUNIT_ASSERT_EQUAL(10, conf.getWorkerThreads());
CPPUNIT_ASSERT(conf.isTrace());