[00:34:07] --- dev-zero@jabber.org has left [01:52:45] --- Russ has left: Disconnected [01:52:57] --- dev-zero@jabber.org_ has left [01:54:33] --- dev-zero@jabber.org has become available [02:20:46] --- dev-zero@jabber.org has left [04:16:11] --- dev-zero@jabber.org has become available [05:57:55] --- mmeffie has become available [06:53:42] --- reuteras has left [07:08:36] how did the NY OpenAFS meetup go last night? [07:09:09] reasonably. fewer ppl than normal. it was q&a rather than a fixed program [07:13:51] sorry I couldn't make it. I had been in NYC 10-11 days already and wanted to get home. [07:42:13] --- Rrrrred has left [07:44:33] --- dwbotsch has become available [07:46:47] --- dwbotsch has left [07:49:28] --- dwbotsch has become available [09:31:16] --- Russ has become available [09:48:40] --- mmeffie has left [13:21:46] --- dlc has become available [13:31:15] --- SecureEndpoints has become available [16:17:56] --- Russ has left: Disconnected [16:32:37] --- Russ has become available [16:38:02] OK, the way rxgen calls cpp needs some revamping. This, for example, is not OK: #ifdef PATH_CPP static char CPP[] = PATH_CPP; #else static char CPP[] = "/lib/cpp"; #endif ... ep = getenv("RXGEN_CPPCMD"); if (ep) strcpy(CPP, ep); else strcpy(CPP, "cl /EP /C /nologo"); [16:40:33] Note the latter fragment is only run in AFS_NT40_ENV, but is still bad. And I want the environment variable thing to work on all platforms, because I need to use it. [17:09:39] jhutz: you want this pulled up? http://www.openafs.org/cgi-bin/wdelta/MAIN/rxgen-cpp-definition-20081221?diff [17:10:18] and then remove the #ifdef AFS_NT40_ENV around the environment block [17:11:15] No. I want the environment thing on non-windows, too. and I want the code that uses it to not assume everything will fit in a particular variable on the stack. I have a patch for you that subsumes that delta and IMHO does better. [17:11:39] Or rather, I will have a patch, once I'm satisfied my build worked. [20:00:21] put it in rt. [20:15:50] --- SecureEndpoints has left [20:16:00] --- SecureEndpoints has become available