IRC log for #brlcad on 20120622

06:06.29CIA-55BRL-CAD: 03Crdueck 07http://brlcad.org * r3945 10/wiki/User:Crdueck/log:
07:34.09CIA-55BRL-CAD: 03anrgmrty * r51280 10/brlcad/trunk/src/conv/g-voxel.c: corrected mistake in calculation of numVoxels in each direction
07:43.33*** join/#brlcad d_rossberg (~rossberg@BZ.BZFLAG.BZ)
08:22.29*** join/#brlcad stas (~stas@188.24.50.251)
10:15.20*** join/#brlcad ksuzee (~ksuzee91@193.151.105.83)
10:36.29*** join/#brlcad merzo (~merzo@174-115-132-95.pool.ukrtel.net)
10:58.21*** join/#brlcad merzo (~merzo@174-115-132-95.pool.ukrtel.net)
11:47.57CIA-55BRL-CAD: 03bob1961 * r51281 10/brlcad/trunk/src/tclscripts/archer/ArcherCore.tcl: Remove unnecessary statement.
11:49.33CIA-55BRL-CAD: 03bob1961 * r51282 10/brlcad/trunk/src/tclscripts/archer/Archer.tcl: Added a statement to insure the temporarily disabled tree columns feature doesn't get enabled via the startup file.
12:42.51*** join/#brlcad stas (~stas@188.24.50.251)
14:26.43CIA-55BRL-CAD: 03Phoenix 07http://brlcad.org * r3946 10/wiki/User:Phoenix/GSoc2012/Reports: /* Week 5 */
14:28.47CIA-55BRL-CAD: 03phoenixyjll * r51283 10/brlcad/trunk/src/librt/ (4 files in 3 dirs): A raw version of brep conversion for part. Some special cases need to be considered later.
15:28.00CIA-55BRL-CAD: 03Plussai 07http://brlcad.org * r3947 10/wiki/User:Plussai/GSoC_2012_log: /* 18 June 2014 */
15:34.19CIA-55BRL-CAD: 03Plussai 07http://brlcad.org * r3948 10/wiki/User:Plussai/GSoC_2012_log: /* 20 June 2014 */
15:55.18CIA-55BRL-CAD: 03bob1961 * r51284 10/brlcad/trunk/src/tclscripts/archer/ArcherCore.tcl: Added ArcherCore::isdouble. This recognizes "." as a double.
16:25.52*** join/#brlcad kane__ (~kane@dsbg-4db57ad7.pool.mediaWays.net)
16:34.56CIA-55BRL-CAD: 03starseeker * r51285 10/brlcad/trunk/src/librt/ (CMakeLists.txt test_botpatches.c): Checkpoint some test code - not terribly useful yet.
16:41.49brlcadstarseeker: how is BRLCAD_ENABLE_OPTIMIZED_BUILD not documented under the CONFIGURATION OPTIONS? :)
16:42.06brlcadif that's missing, wonder what else is missing... :)
16:42.33brlcador was it just because it was mentioned earlier in INSTALL that it got back the check
16:43.25starseekeruh... I might have been thinking that one is normally set/unset by the build type (Debug/Release)
16:44.09brlcadmore interestingly...
16:44.14starseekeroh no...
16:44.16brlcadCMake Warning: The variable, 'BRLCAD_ENABLE_OPTIMIZED_BUILD', specified manually, was not used during the generation.
16:44.55starseekerwhere are you getting that from, bty?  I only see BRLCAD_OPTIMIZED_BUILD
16:45.23brlcadINSTALL
16:45.46starseekerah, yeah
16:45.53starseekerthat changed
16:45.57starseekercrud...
16:46.05starseekerthat part of the text is not autosynced
16:46.09starseekerfixes...
16:46.18brlcadwasn't there a regression that checked to make sure INSTALL listed everything?
16:46.52starseekeronly for autotools - for most of the user visible options in CMake, the docs are auto-appended to INSTALL
16:47.33brlcadblinks
16:47.38brlcadthat should effectively make sure INSTALL lists everything
16:48.02starseekerBRLCAD_OPTIMIZED_BUILD isn't normally user visible
16:48.08starseekerit's wrapped up in the build type
16:48.24starseekerarguably, I shouldn't be discussing it there
16:49.17brlcadyet debug and warnings are?
16:49.33starseekeractually, there is a user visible BRLCAD_FLAGS_OPTIMIZATION
16:51.14brlcadso the issue that came up, the use case, is that I wanted a quick distcheck with what I knew would generate the most warnings -- enable-all src/others, enable debug (which I happen to know is default), and optimized compile
16:51.15starseekerhmm... yeah, that's not one of the auto-documented ones
16:51.34brlcadso I quick-scan INSTALL for optimization .. find that flag, no others, and .. here we are
16:51.43starseekermake -j9 distcheck will do both enable-all release and enable-all debug
16:51.59brlcadI know there's a way to get it with the release modes
16:53.31brlcadthe distcheck is really a non-sequitor -- I was attempting a fairly standard goal, read the docs, and was left hanging :)
16:53.51brlcadbasically a documentation bug, just trying to figure out what the fix needed it
16:53.51starseekernot arguing the docs are incorrect - just making sure I've got the right option before fixing
16:54.16starseekershould be BRLCAD_ENABLE_OPTIMIZED_BUILD -> BRLCAD_FLAGS_OPTIMIZATION
16:54.25brlcadokay, so sounds like three issues
16:55.19brlcadthat little explanation snippet (which I agree is debatable to talk about that there)
16:56.02CIA-55BRL-CAD: 03starseeker * r51286 10/brlcad/trunk/INSTALL: This option name has changed
16:56.29brlcadit either needs fixing or moving/removing .. if removed, then INSTALL probably should talk about the build types there instead
16:58.29brlcadI personally would have expected to see independent debug and optimization options in the configuration options section because they go hand in hand
17:00.15CIA-55BRL-CAD: 03starseeker * r51287 10/brlcad/trunk/CMakeLists.txt: tweak
17:00.15brlcadthe release mode is a timesaver, bundling options together into a configuration profile
17:00.15starseekershrugs - my thinking was probably that it was more likely to want to fiddle with debug/optimization than the other settings...
17:00.15starseekerso they're documented in that first bit
17:00.20brlcadyou mean more likely to fiddle debug than optimization?
17:00.37starseekerno, sorry - both of those as opposed to turning on/off individual libs
17:00.41starseekeror Tk or something
17:00.54brlcadoh I agree
17:01.12brlcadthat's why I found it surprising that optimization was missing, especially if that section is auto-generated
17:01.16starseekerthat's why the optimization isn't one of the auto-documented options pulling their descriptions from the CMake logic - it's more "integrated"
17:01.22starseekerthat part isn't
17:01.50starseekerauto-generation starts after CONFIGURATION OPTIONS
17:03.09starseekerI guess I figured people would read "installing from source" prior to going for the config options section, but looks like that experiment just failed in a real test
17:03.21brlcadright
17:03.40brlcadexcept you did add debug
17:03.56brlcadso maybe just oversight that opt is missing?  some flag that gets set on the var?
17:04.01starseekerthat's a real problem then - either we annoyingly duplicate a lot of content in multiple places, or we put everything in the configuration options mechanism
17:04.32starseekerah, the debug flags is there - I hadn't realized that
17:04.57brlcadso specifically with the optimization option, what's funny is that the writeup basically says "you can use -DBRLCAD_OPTIMIZED_BUILD=ON to turn on optimization, but see the CONFIGURATION OPTIONS section for a description of ALL options"
17:05.05starseekermaybe that's the solution then - take it out of where it is, and make it do the configuration options thing
17:05.24brlcadso I jumped down, and of course didn't find it there :)
17:05.56starseekeryeah - that was me thinking that "you'll probably want speed, so here's that option, and for everything else look below"
17:06.24starseekerin case you haven't noticed yet, writing these was rather a hurried affair
17:06.50starseekernot to mention my often odd notions of "flow" :-P
17:07.00brlcadI've tried not to look too closely given the push for this big release :)
17:07.06starseekerhehe
17:07.17starseekerlet me fix that one quick
17:07.32starseekerbut remember, we're not syncing to stable
17:07.34brlcadI'd leave the duplication for now until the flow is sorted
17:07.45starseekerdo you want me to do an individual sync of that one change?
17:08.20brlcadI'd just copy the latest version of the file and let it merge later
17:09.05starseekereh?  you mean manually modify it for the release tarballs?
17:11.24brlcadno, I mean I intend to cp trunk's INSTALL and NEWS files to STABLE
17:11.35brlcadand/or any others
17:11.36starseekerah
17:11.49brlcadthey'll merge cleanly if done in the right order
17:11.56brlcadthe next time it's sync'd
17:12.08starseekerit'll be a couple more than that, probably
17:12.13starseekerhang on a sec...
17:12.20brlcadcouple more what?
17:12.24starseekerfiles
17:12.44brlcadwell, the version files, something else?
17:12.47starseekerCMakeLists.txt, at a minimum, if we're adding BRLCAD_FLAGS_OPTIMIZATION to the CONFIGURATION OPTIONS section
17:12.59brlcadthat's not a problem
17:13.23starseekerit may go deeper - that flag supports the "AUTO" mechanism, and I'm not sure if the others do
17:13.25brlcadwhat would that be needed for?
17:13.44starseekeryou wanted the optimization documention to be with the other options
17:14.07brlcadyes...
17:14.33brlcadthat's needed, but the updated CMakeLists.txt doesn't exactly need to be on STABLE, its product (INSTALL) does
17:14.59starseekerthere's an internal check that the INSTALL file in the source tree corresponds to the one generated by CMake
17:15.19starseekerif STABLE's INSTALL isn't in sync with the one generated by the CMake build logic in install, it'll complain (loudly)
17:15.35starseekersorry, CMake build logic in STABLE
17:15.35brlcadokay, no biggie either way
17:15.42brlcadanything else?
17:16.08brlcadI'm not worried about distcheck failing at this point unless sources get merged or new files are added
17:16.12starseekerI don't know yet - if BRLCAD_OPTION (the one that supports documentation) also can handle AUTO, we're good.  If not, I'll have to expand it
17:16.28brlcaddistcheck passed, so this is just documentation cleanup for tarball tagging
17:16.31starseekerit's not a distcheck failure, it's a warning on CMake configure
17:16.37brlcadew
17:16.54brlcadwhy would a user care about that? :)
17:17.00starseekerthat was deliberate, to make sure we couldn't ignore the docs when editing
17:17.09starseekerthe user wouldn't normally see it
17:17.28starseekerbut a developer making changes would
17:17.38brlcadyeah, except that's akin to making configure run some of the distcheck steps
17:18.09brlcadsure, user wouldn't normally see it .. because we'll catch it .. but that's not the point ;)
17:18.12starseekershrugs - configure also complains if you try to use the distcheck mechanisms to ignore a file that doesn't exist
17:18.16brlcadit's fine, just really really odd
17:18.39starseekerI just figured the more annoying it was to NOT fix the docs, the better shot we'd have that they'd STAY fixed
17:18.54brlcadyou could argue that for all the distcheck options
17:19.06starseekeryeah, but a lot of them are time expensive
17:19.06brlcadand style
17:19.08brlcadand ws
17:19.09starseekerthose aren't
17:19.37starseekerwe don't have a programmatic way to check style and ws, or I'd probably be on board with that
17:19.48brlcadactually we do, at least partially
17:20.17starseekeryou mean running the shell scripts on all the files?
17:20.46brlcadstill, not a problem at all .. just want to be conscious and aware that it is odd and might cause us issues
17:21.04brlcadit'd be hipocritical to consider it a problem
17:21.47starseekerit's easy to turn off...  CMakeLists.txt around line 2180
17:21.58starseekerjust comment out the WARNING
17:23.35brlcadnah, I actually like enforcement like that .. just really odd :)
17:24.12starseekerbrlcad: I'd advise for now that we just go with the fixed OPTIMIZATION variable, and I'll figure out how to get the optimization variable in with the rest of them for the next release
17:24.46brlcadok
17:24.55starseekershudders... next you'll be liking the idea of turning on DocBook struct... talk about carnage
17:25.02starseekerstrict even
17:25.54starseekerAlthough... Tom has been doing a lot of man page work lately...
17:26.00starseeker<evil grin>
17:26.22brlcadhm? DocBook?
17:26.26brlcadnot for INSTALL
17:26.34starseekeryeah, separate isuse
17:26.35brlcadthe top-level docs should be plain text
17:26.36starseekerissue
17:27.02starseekeryeah - was just running through the "what else can we turn on to check things automatically" list
17:27.17brlcadoh, *strict* docbook
17:27.19brlcadgotcha
18:15.44*** join/#brlcad Al_Da_Best (~Al_Da_Bes@5e0e150d.bb.sky.com)
18:32.45*** join/#brlcad Mahi (~Mahi@li364-30.members.linode.com)
19:22.42*** join/#brlcad cristina (~quassel@188.24.50.251)
19:41.19CIA-55BRL-CAD: 03brlcad * r51288 10/brlcad/trunk/src/other/libpng/configure.ac: need AC_PROG_LIBTOOL for LT_INIT else we can get an error about LIBTOOL being undefined.
20:02.05CIA-55BRL-CAD: 03bob1961 * r51289 10/brlcad/trunk/src/tclscripts/lib/TkTable.tcl: Updated the TkTable::keyVisible and TkTable::handleKey methods to do a better job of recognizing visible keys.
20:08.57*** part/#brlcad kane__ (~kane@dsbg-4db57ad7.pool.mediaWays.net)
20:27.44CIA-55BRL-CAD: 03bob1961 * r51290 10/brlcad/trunk/src/tclscripts/ (archer/ArcherCore.tcl lib/Ged.tcl): Moved the ArcherCore::isdouble functionality to cadwidgets::Ged.
20:48.18CIA-55BRL-CAD: 03bob1961 * r51291 10/brlcad/trunk/src/tclscripts/lib/Ged.tcl: Minor mod to cadwidgets::Ged::isdouble.
20:49.01CIA-55BRL-CAD: 03brlcad * r51292 10/brlcad/branches/STABLE/ (6 files in 4 dirs): manually merge a few last-minute changes from trunk. fix install docs, few mged scripting robustness checks, and command blather.
21:01.06*** join/#brlcad andrei_ (~andrei@5-12-65-148.residential.rdsnet.ro)
21:10.15CIA-55BRL-CAD: 03Stattrav 07http://brlcad.org * r3949 10/wiki/User:Stattrav/GSoC2012_log: Updation of the logs.
21:26.47CIA-55BRL-CAD: 03r_weiss * r51293 10/brlcad/trunk/src/tclscripts/mged/grouper.tcl:
21:26.47CIA-55BRL-CAD: Updated the mged gr/grouper command in file grouper.tcl. Fixed a bug where the
21:26.47CIA-55BRL-CAD: display state was not reset when grouper exited. This was causing the frame
21:26.47CIA-55BRL-CAD: buffer to not display correctly. Improved logic to ignore a selection box with
21:26.47CIA-55BRL-CAD: zero area. Changed the output of the contents of the selection box to list on a
21:26.47CIA-55BRL-CAD: single line. This change was to make easier copying and pasting of this list.
21:26.48CIA-55BRL-CAD: Improved status messages while grouper is processing.
21:37.25CIA-55BRL-CAD: 03bob1961 * r51294 10/brlcad/trunk/src/tclscripts/archer/ (ArcherCore.tcl GeometryEditFrame.tcl ShaderEdit.tcl): Using ::cadwidgets:Ged::isdouble.
21:37.53CIA-55BRL-CAD: 03starseeker * r51295 10/brlcad/trunk/ (3 files in 3 dirs): make bots instead of plot files.
21:53.14CIA-55BRL-CAD: 03bob1961 * r51296 10/brlcad/trunk/src/tclscripts/lib/Ged.tcl: Putting back a line of code that accidentally got clobbered.
21:56.51*** join/#brlcad ksuzee (~ksuzee91@193.151.105.83)
21:57.21*** part/#brlcad ksuzee (~ksuzee91@193.151.105.83)
22:04.12*** join/#brlcad ksuzee (~ksuzee91@193.151.105.83)
22:04.22*** part/#brlcad ksuzee (~ksuzee91@193.151.105.83)
22:55.56*** join/#brlcad ibot (~ibot@rikers.org)
22:55.56*** topic/#brlcad is BRL-CAD Open Source Solid Modeling || http://brlcad.org || logs: http://ibot.rikers.org/%23brlcad/ || Congratulations to our 11 accepted GSoC students!!

Generated by irclog2html.pl Modified by Tim Riker to work with infobot.