| 00:00.51 | *** join/#brlcad merzo (~merzo@140-4-133-95.pool.ukrtel.net) | |
| 00:05.10 | mpictor | starseeker: I think this is different - it's not in C_FLAGS or the target's properties |
| 00:05.16 | mpictor | \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ type/SdaiDatum_or_common_datum.cc |
| 00:05.28 | starseeker | hmm |
| 00:05.29 | mpictor | ^^^ that's the line in the makefile |
| 00:05.40 | mpictor | https://gist.github.com/mpictor/8677627#file-build-make-L58 |
| 00:06.07 | mpictor | I asked on #cmake but haven't got a response yet |
| 00:07.29 | mpictor | I hate to file a cmake bug without winnowing it down to a few lines of code first |
| 00:07.37 | starseeker | nods |
| 00:09.24 | mpictor | cmake's set() ignores extra spaces around the args, right? |
| 00:09.26 | starseeker | mpictor: what's the git clone line to get the repo with this problem? |
| 00:09.29 | mpictor | that's all I can think of |
| 00:09.44 | starseeker | usually... is there a reason the spaces are there? |
| 00:10.20 | mpictor | right here https://gist.github.com/mpictor/8677627#file-command-line-sh-L6 :) |
| 00:10.39 | mpictor | the spaces are so the file is more human readable |
| 00:10.39 | starseeker | ah, sorry |
| 00:10.45 | mpictor | np |
| 00:10.48 | starseeker | nifty trick with the line highlighting |
| 00:10.50 | mpictor | lots to scroll past |
| 00:11.05 | mpictor | yea, I think github allows that everywhere |
| 00:19.45 | mpictor | starseeker: the cmakelists is created at https://github.com/stepcode/stepcode/blob/mp/config-parse/cmake/schema_scanner/schemaScanner.cc#L150 |
| 00:20.22 | starseeker | ah, so it's autogenerated |
| 00:20.23 | mpictor | and the lists of files are created in this switch statement: https://github.com/stepcode/stepcode/blob/mp/config-parse/cmake/schema_scanner/schemaScanner.cc#L233 |
| 00:34.31 | starseeker | weird that it's only the clean rule |
| 00:36.19 | mpictor | it's not just 'make clean' for me, that just runs into an error faster |
| 00:51.41 | starseeker | my best guess at the moment is the SCHEMA_TARGETS macro is interacting badly with the generated list in some fashion... |
| 00:53.50 | mpictor | I hadn't thought of that |
| 00:54.29 | mpictor | by the way, SC's main CMakeLists is much smaller now - I moved a whole bunch of stuff into files in cmake/ that I include |
| 00:54.51 | starseeker | nods - good call for readibility |
| 00:55.23 | starseeker | has bits of BRL-CAD's CMake that he keeps re-using and eventually made into includeable files - need to feed those back into BRL-CAD's build |
| 00:56.52 | starseeker | the main question in my mind right now is *why* all the back-slashes behind type/SdaiDatum_or_common_datum.cc |
| 00:57.23 | starseeker | I don't see anything in the initial list definitions that looks weird... |
| 01:04.59 | starseeker | mpictor: does this fix it? http://paste.lisp.org/display/141063 |
| 01:18.27 | mpictor | something funny going on with that site... the 3rd '+' line is truncated |
| 01:18.53 | starseeker | grr |
| 01:18.59 | mpictor | and when I try to copy, blank lines contain '.', even on raw |
| 01:19.14 | mpictor | pretty sure that line ended with endl |
| 01:19.16 | starseeker | ok, I'll try a different one |
| 01:19.19 | mpictor | I'm trying it now |
| 01:19.25 | mpictor | think I got it |
| 01:20.14 | starseeker | http://pastebin.mozilla.org/4124352 |
| 01:20.18 | starseeker | that work any better? |
| 01:20.36 | starseeker | oh wait |
| 01:20.39 | starseeker | it's on my end |
| 01:20.57 | mpictor | I got it :) |
| 01:21.29 | mpictor | long lines are getting truncated by your browser? |
| 01:21.36 | starseeker | no, my terminal |
| 01:21.36 | mpictor | that would be incredibly frustrating! |
| 01:21.41 | mpictor | ohhh |
| 01:21.49 | starseeker | fixed now http://pastebin.mozilla.org/4124373 |
| 01:22.13 | mpictor | 'make clean' works now |
| 01:22.46 | mpictor | I'm getting an error with some CMake stuff that hadn't ever had a chance to run before |
| 01:22.52 | starseeker | working on it now |
| 01:27.40 | starseeker | http://pastebin.mozilla.org/4124450 |
| 01:28.11 | starseeker | whoops - one second |
| 01:29.25 | mpictor | in that patch, _res is no more |
| 01:29.31 | starseeker | got it http://pastebin.mozilla.org/4124456 |
| 01:29.45 | starseeker | mpictor: that was the "whoops" ;-) |
| 01:29.47 | starseeker | fixed |
| 01:30.28 | mpictor | heh |
| 01:30.39 | starseeker | that seems to work here |
| 01:30.46 | mpictor | I'll try it... |
| 01:31.38 | mpictor | patch unexpectedly ends in middle of line |
| 01:31.54 | starseeker | grr |
| 01:31.57 | starseeker | one more time... |
| 01:32.52 | starseeker | that's what git diff is giving me... |
| 01:33.09 | starseeker | http://pastebin.mozilla.org/4124478 |
| 01:34.47 | starseeker | there's one long line that might be wrapping on you if you're copy/pasting... |
| 01:35.26 | starseeker | does the file from http://pastebin.mozilla.org/?dl=4124478 have the same issue? |
| 01:35.49 | mpictor | patch is still complaining, I'll apply it by eyeball ;) |
| 01:36.03 | starseeker | huh - sorry |
| 01:36.56 | starseeker | it may not be totally necessary to use "SDIR" instead of "DIR" - I just avoided the latter because I wasn't sure if it would end up conflicting with some internal variable or some such |
| 01:38.59 | starseeker | foolhardily enables all schemas and watches in awe as his CPU cores are consumed... |
| 01:39.15 | mpictor | hah |
| 01:39.52 | mpictor | surely not as bad as that torture test multi-config brlcad build you suggested I try |
| 01:40.07 | starseeker | hehe |
| 01:40.14 | starseeker | probably true |
| 01:40.36 | starseeker | in this case the awe is as much for the breakout of the schemas into nice individual pieces as the CPU crunching |
| 01:40.56 | starseeker | 9 instances of the full 16 meg AP242 would probably just *melt* this machine |
| 01:41.37 | mpictor | now that I'm generating CMakeLists, cmake runs *much* faster |
| 01:41.49 | mpictor | and there are no 41MB Makefiles :D |
| 01:42.02 | starseeker | always a good thing |
| 01:42.06 | mpictor | yes |
| 01:42.26 | starseeker | I gotta hand it to you Mark - that's a creative approach |
| 01:42.32 | mpictor | before, I thought that it was slow because of scanning all those headers... but at that point, the headers don't exist to be scanned |
| 01:42.39 | mpictor | well thank you :) |
| 01:42.59 | starseeker | has used CMake to generate a lot of files, but never used CMake plus a generator to generate more CMake files |
| 01:43.39 | starseeker | meta-evil :-) |
| 01:43.51 | mpictor | yes you have... just now |
| 01:43.54 | mpictor | LOL |
| 01:44.21 | mpictor | less evil than generating autotools files! |
| 01:44.34 | starseeker | shivers |
| 01:46.10 | starseeker | autogenerates a "configure" script that maps autotools syntax to CMake options and for a while had to trigger the old autotools build from CMake for distribution testing - hopefully that's as far back into autotools as I'll ever have to go :-) |
| 01:47.03 | starseeker | m4 + shell + Make + stuff on top of make + ... |
| 01:47.04 | mpictor | bleh |
| 01:47.20 | starseeker | it's like they were evolving towards a significant subset of Perl |
| 01:47.39 | starseeker | if you want to be that confusing, why not just go straight to the source? |
| 01:47.49 | mpictor | no kidding |
| 01:48.56 | starseeker | considers IOCCC to be, in a backhanded sort of way, praise for C |
| 01:49.26 | starseeker | in C it's an achievement to make something useful and hard to understand |
| 01:49.47 | mpictor | yeah |
| 01:49.55 | starseeker | is still convinced you could pipe random line noise into a Perl interpreter and wait for something useful to happen |
| 01:50.43 | starseeker | 68% and counting |
| 01:51.33 | starseeker | getting down to the bigger schemas now |
| 01:51.50 | starseeker | mpictor: did a manual application of those changes work for you? |
| 01:52.05 | mpictor | I got distracted, sorry |
| 01:52.35 | starseeker | np |
| 01:55.38 | mpictor | I added that cmake message about missing headers because several people asked on the mailing list |
| 01:55.48 | mpictor | I guess people aren't used to seeing that on windows |
| 01:56.45 | mpictor | did it print out a whole bunch of file names for you? |
| 01:59.31 | starseeker | um |
| 01:59.54 | starseeker | I don't recall seeing a big list, but I may have missed it in all the compilation output |
| 02:02.32 | mpictor | it was when it said "scanning dependencies for ..." |
| 02:02.48 | mpictor | I don't remember seeing it when I was building a single small schema |
| 02:03.06 | mpictor | ought to be easy enough for me to find |
| 02:03.12 | mpictor | thanks for your help! |
| 02:03.14 | starseeker | I see the message |
| 02:03.37 | mpictor | I'm still at 56% |
| 02:04.06 | mpictor | hmmm, I just saw another "scanning dependencies..." without a list of files |
| 02:04.30 | starseeker | here's what I'm seeing: http://pastebin.mozilla.org/4124641 |
| 02:06.20 | mpictor | oh, I mean while it's compiling |
| 02:06.26 | starseeker | ah |
| 02:16.40 | mpictor | saw it again and managed to copy/paste in time |
| 02:16.41 | mpictor | http://pastebin.mozilla.org/4124743 |
| 02:17.10 | mpictor | grep isn't finding any lines that look like they'd print that message |
| 02:17.20 | mpictor | maybe CMake is doing it? |
| 02:17.29 | starseeker | the Generating messages? |
| 02:17.32 | starseeker | yeah, that's CMake |
| 02:17.34 | mpictor | yes |
| 02:17.59 | starseeker | Pretty sure that's normal - I think there's a way to override it though if you'd like a less verbose message |
| 02:18.34 | mpictor | there must be, I don't remember files being printed before |
| 02:19.10 | mpictor | time for me to hit the sack. thanks for your help! |
| 02:19.20 | starseeker | my pleasure |
| 02:19.34 | starseeker | I'll post a less verbose message solution to the list if I find it |
| 02:19.45 | mpictor | sounds good |
| 02:48.35 | starseeker | done and posted |
| 02:48.38 | starseeker | yawns |
| 02:49.56 | starseeker | mpictor: would probably be interesting to compare the build times for all schemas without multiple files and the new setup on a machine with a lot of CPU cores |
| 04:17.25 | *** join/#brlcad frankbutt (~frankbutt@66.172.11.32) | |
| 04:17.27 | *** part/#brlcad frankbutt (~frankbutt@66.172.11.32) | |
| 04:48.24 | *** join/#brlcad kesha (~kesha@14.139.122.114) | |
| 05:09.38 | *** join/#brlcad FOSScookie (~brian@107-200-34-31.lightspeed.tulsok.sbcglobal.net) | |
| 08:25.13 | *** join/#brlcad FOSScookie (~brian@107-200-34-31.lightspeed.tulsok.sbcglobal.net) | |
| 09:27.57 | *** join/#brlcad Ch3ck_ (~Ch3ck@195.24.220.134) | |
| 09:55.05 | *** join/#brlcad Ch3ck__ (~Ch3ck@195.24.220.134) | |
| 12:56.03 | *** join/#brlcad Anaphaxeton (~george@unaffiliated/anaphaxeton) | |
| 13:08.30 | *** join/#brlcad FreezingCold (~FreezingC@205.211.52.10) | |
| 13:17.50 | *** join/#brlcad KimK (~Kim__@ip24-255-223-153.ks.ks.cox.net) | |
| 14:32.50 | *** join/#brlcad FreezingCold (~FreezingC@205.211.50.10) | |
| 14:52.45 | *** join/#brlcad FreezingCold (~FreezingC@205.211.52.162) | |
| 14:56.01 | *** join/#brlcad chick (~chick@195.24.210.66) | |
| 15:27.36 | *** join/#brlcad chick (~chick@195.24.210.66) | |
| 15:34.27 | Notify | 03BRL-CAD:carlmoore * 59557 brlcad/trunk/src/conv/step/g-ap242/AP242_managed_model_based_3d_engineering_20131030.exp: remove a large number of trailing blanks/tabs |
| 15:53.31 | Notify | 03BRL-CAD:starseeker * 59558 brlcad/trunk/src/conv/step/g-step/g-step.cpp: Use db_ls to make specifying an object in the .g file optional |
| 15:58.58 | Notify | 03BRL-CAD:tbrowder2 * 59559 (brlcad/trunk/include/bu.h brlcad/trunk/src/libbu/bitv.c and 2 others): include/bu.hsrc/libbu/bitv.c+ add another arg to bu_binary_to_bitv to allow user to specify aminimum byte length+ adding new bitv compare function which does not use lengthssrc/libbu/tests/CMakeLists.txtsrc/libbu/tests/bu_bitv.csrc/libbu/tests/test_funcs.csrc/libbu/tests/test_internals.h+ add bitv/binary/compare |
| 15:59.00 | Notify | tests+ add a debug bitv dump function as a local library |
| 16:04.06 | Notify | 03BRL-CAD:starseeker * 59560 brlcad/trunk/src/conv/step/g-step/CMakeLists.txt: Given how this logic is set up, need a slight re-think to handle a tops list. Start reorg |
| 16:26.31 | *** join/#brlcad Ch3ck_ (~Ch3ck@195.24.220.134) | |
| 16:39.58 | Notify | 03BRL-CAD:n_reed * 59561 brlcad/trunk/src/libbu/bitv.c: warning fixes: signed/unsigned comparison, shadowed name, unused |
| 16:53.27 | Notify | 03BRL-CAD:n_reed * 59562 (brlcad/trunk/src/conv/step/g-step/Assembly_Product.cpp brlcad/trunk/src/conv/step/g-step/Assembly_Product.h and 2 others): need to rename includes to match renamed file |
| 17:03.10 | *** join/#brlcad FreezingCold (~FreezingC@205.211.52.10) | |
| 17:44.05 | Notify | 03BRL-CAD:n_reed * 59563 brlcad/trunk/src/libbrep/boolean.cpp: move logic to build trimmed face array to function; make one call per brep instead of doing them both at once |
| 18:24.50 | Notify | 03BRL-CAD:starseeker * 59564 (brlcad/trunk/src/conv/step/g-step/Assembly_Product.cpp brlcad/trunk/src/conv/step/g-step/CMakeLists.txt brlcad/trunk/src/conv/step/g-step/Trees.cpp): Start reworking logic. Split the wrapper/non-wrapper test out into its own independent function that doesn't need search logic. |
| 18:33.31 | Notify | 03BRL-CAD:n_reed * 59565 brlcad/trunk/src/libbrep/boolean.cpp: simplify nested for loops, pulling second nested loop into separate function |
| 18:39.32 | Notify | 03BRL-CAD:starseeker * 59566 (brlcad/trunk/src/conv/step/g-step/Assembly_Product.cpp brlcad/trunk/src/conv/step/g-step/Comb.cpp brlcad/trunk/src/conv/step/g-step/Comb.h): Fix build, add convenience function |
| 18:46.15 | Notify | 03BRL-CAD:starseeker * 59567 brlcad/trunk/src/conv/step/g-step/Comb.cpp: Fix comments, simplify |
| 19:04.36 | Notify | 03BRL-CAD:n_reed * 59568 brlcad/trunk/src/libbrep/boolean.cpp: move categorization of trimmed faces as belonging/not belonging to final brep to separate function |
| 19:18.17 | *** join/#brlcad FreezingCold (~FreezingC@205.211.54.161) | |
| 20:02.01 | *** join/#brlcad nick__ (b02a52ff@gateway/web/freenode/ip.176.42.82.255) | |
| 20:06.29 | Notify | 03BRL-CAD:indianlarry * 59569 brlcad/trunk/src/libbrep/PullbackCurve.cpp: Initial pullback cleanup but still work in progress. Working on resolving issues generating trimming curves where the UV pullbacks of the 3D curve cross a closed surface seam. |
| 20:43.20 | Notify | 03BRL-CAD:n_reed * 59570 brlcad/trunk/src/libbrep/boolean.cpp: Make trimmed face categorization more readable by 1) renaming is_face_inside_brep to face_brep_location and having it throw exceptions and return constant names instead of returning int, and 2) separating the debug messages from the normal code. |
| 21:03.37 | *** join/#brlcad merzo (~merzo@140-4-133-95.pool.ukrtel.net) | |
| 21:10.53 | Notify | 03BRL-CAD:starseeker * 59571 (brlcad/trunk/src/conv/step/g-step/Assembly_Product.cpp brlcad/trunk/src/conv/step/g-step/CMakeLists.txt and 9 others): Rename G_STEP_internal and it's cpp file |
| 21:11.35 | Notify | 03BRL-CAD:starseeker * 59572 brlcad/trunk/src/conv/step/g-step/AP203.cpp: Now that we've moved the file, fix it. |
| 21:44.54 | Notify | 03BRL-CAD:starseeker * 59573 (brlcad/trunk/src/conv/step/g-step/AP203.h brlcad/trunk/src/conv/step/g-step/CMakeLists.txt and 7 others): Add some more per-type headers, reorganize includes a bit. |
| 22:23.25 | Notify | 03BRL-CAD:starseeker * 59574 (brlcad/trunk/src/conv/step/g-step/AP203.cpp brlcad/trunk/src/conv/step/g-step/AP203.h and 19 others): Move some common parameters (and a few things that needed to be more commonly available and weren't) to a common structure to be passed between functions. |
| 22:31.46 | Notify | 03BRL-CAD:r_weiss * 59575 brlcad/trunk/CMakeLists.txt: update to cmake settings to fix windows 32bit build |
| 22:45.35 | Notify | 03BRL-CAD:starseeker * 59576 brlcad/trunk/src/conv/step/g-step/g-step.cpp: Whoops - need to update the nref stuff before DB_LS_TOPS will work. |
| 22:46.30 | Notify | 03BRL-CAD:starseeker * 59577 brlcad/trunk/src/conv/step/g-step/Trees.cpp: Now that we've got access to the maps, make sure we don't dupliate work when handling multiple combs. |
| 22:51.21 | *** join/#brlcad FOSScookie (~brian@107-200-34-31.lightspeed.tulsok.sbcglobal.net) | |
| 23:22.23 | Notify | 03BRL-CAD:starseeker * 59578 (brlcad/trunk/src/conv/step/g-step/Trees.cpp brlcad/trunk/src/conv/step/g-step/Trees.h): Tweaks to comments |
| 23:24.08 | Notify | 03BRL-CAD:starseeker * 59579 brlcad/trunk/src/conv/step/g-step/Trees.cpp: Trim header list |