#foswiki 2015-04-10,Fri

↑back Search ←Prev date Next date→ Show only urls(Click on time to select a line by its url)

WhoWhatWhen
***ChanServ sets mode: +o gac410 [00:09]
jomohi gac410 :) [00:13]
gac410howdy jomo [00:13]
jomojust tried to test the beta - the new configure is very very bugged, bad and unusable - just added an new task-report. ;(
http://foswiki.org/Tasks/Item13364
[00:15]
gac410Hi Jomo, Some of this has already been fixed. The pink error box undefined subroutine ... was a bug where configure expected all set operations to be character strings, but when called internal to the ext. installer, it gets perl structures. Fixed in Item13357 [00:21]
FoswikiBothttp://foswiki.org/Tasks/Item13357 [ Item13357: Configure issues installing older extensions ] [00:21]
gac410The useless dialog - wizard active. was a half hearted attempt to let the user know that something is running. Since the interface isn't modal, we needed some feedback that something was happening, [00:22]
jomoah soo, but probably it isn't included in the beta.tar.gz yet... [00:23]
gac410NatSkin is probably the worst extension to install. You sure gave it the "Acid test" :) Maybe FugueIconsContrib is worse (copies in >7000 files takes 5-10 minutes.
All the Red !, This is also an known issue. They *might* have been cleared once you reloaded the configuration. There are some issues getting the "Spec" of config variables picked up in the browser DOM.
[00:23]
jomoBut the NatSkin and the MetaCommentPlugin (IMHO) will be the most wanted extensions :) - NatSkin because it is (besed on screenshots) somewhat nicer as the default patternSkin... - So, if the NatSkin will get installed without problems - probably avarything will went OK ;)
Yeah, if you check the attached 80 pages report - here is many "errors" about missing .spec files.
[00:26]
gac410True. Actually that extension installer has not changed at all between 1.1.9 and 1.2.0. With the exception that it enables the extensions now.
http://foswiki.org/Download/FoswikiRelease01x02x00Beta1#Known_issues has a manually kept list of what we've found, and will hopefully have fixed in the next beta
gac410 goes back to reread that 80 page report :)
I think those Spec errors might be because the extension installation was "Simulated"
Ah damn. FilterPlugin installs lib/CPAN/lib files. :P
[00:27]
jomothe new JSONRPC could be an step forward, but wondering why the developer didn't used something what is already done, (at least partially), e.g. angularjs (for the single-page-config), some nice json->form libraries like: http://schemaform.io and such - nice, repsonsive design and such... [00:32]
gac410gac410 doesn't do why questions :)
Some of the other issues you reported, have tasks, but just didn't get done for 1.2.0 Item11834 for ex.
[00:32]
FoswikiBothttp://foswiki.org/Tasks/Item11834 [ Item11834: Hide the details in the extension installation report. ] [00:36]
jomoyeah, its mine .. ;) - but now i have a feeling than it is NOW much more verbose... ;) [00:37]
gac410No... no changes really to Package.pm At least nothing significant.
Actually it is much LESS verbose. Under 1.1.9, it would have installed each dependency every time it is mentioned. In some cases installing some extensions 4-5 times.
[00:37]
jomook ;) - so, when 1.) after the fres FW installation 2.) the configure could install NatSkin without errors - probably the foswiki will be ship-able ;) ;) [00:38]
gac410yeah. I'll give it a test sometime. First though I want to profile Package.pm The install takes much to long. I'm not sure why it takes so much time.
That one is Item13349 Also a 1.1.9 issue. no better on 1.2.0
[00:39]
FoswikiBothttp://foswiki.org/Tasks/Item13349 [ Item13349: Installation produces perl code on screen. ] [00:41]
gac410(It was really javascript code, and I think it was because apache timed out while perl code was copying 7000 files. [00:41]
jomoyeah - probably refactoring into css-sprites is too much work ;)
(e.g. one file with 7000 images - + the css for the positions)
[00:42]
gac410right I understand. ... Images are a 3rd party icon set. I just want configure to not choke on it. [00:43]
jomoah - understand. ;) [00:44]
gac410I couldn't even profile it. NYTProf slowed it down, 20 minutes later I killed it. [00:45]
jomopeoples really uses those icons? asking because myself never used any icon - and now using font-awesome for all my icon-needs.. ;) [00:45]
gac410No idea. I've never installed it... until trying to figure out the crash. There are lots of extensions out there I've never touched. [00:45]
jomo:)
some cosmetic things around - for example, the installation guide pointed the user to "apacheConfigGenerator" but in it missing the 1.2 version (using 1.1 works, but it is again confusing for the new users) - but such mini-problems arent importatnt now...
[00:46]
gac410There is a "Test" Foswiki:Support.ApacheConfigGenerator24 which will replace the ACG I think it includes 1.2. [00:49]
FoswikiBothttp://foswiki.org/Support.ApacheConfigGenerator24 [ ApacheConfigGenerator24 ] [00:49]
gac410Well MimeIconPlugin is probably as bad as FugueIconsContrib That's most of your output. :) [00:50]
jomo;) yeah - and many lines such lib/CPAN/lib/Text/Unidecode/x15.pm - don't understand why they needed... /hm, again one WHY question :)/ [00:51]
gac410No idea on that one. Thats part of bundled Text::Unidecode cpan package. Need to ask that one on metacpan or p5p or wherever [00:54]
jomoanother: My english isn't the best one - so the following is maybe confusing only for me, but the text such: Foswiki::Plugins::PageOptimizerPlugin version >=0.10 required -- perl module is not installed - Optional - Optional dependency, will not be automatically installed - is confusing for me - The module is required OR it is optional? ;) [00:54]
FoswikiBothttp://trunk.foswiki.org/System/PerlDoc?module=Foswiki::Plugins::PageOptimizerPlugin [00:54]
gac410Yeah. Some of that is because of the way the comments in the DEPENDENCIES files are strung together.
The first word "required" should maybe be changed to "requested" "needed", "examined" , "probed" ??? Not sure how best to say that we are looking for it.
[00:55]
jomohm... simply don't print it... this is importatnt only for developers - the user simply want press the red button, and the app will do what i mean ;) - I really don't need/want to know about the module versions... i'm an user (at least until not going to hack foswiki :) ;)
unnecessary and unimortatnt informatioin overload is always bad. ;)
[00:59]
gac410hm. well if the requirement is for 1.2, and only 1.1. is installed, the user needs to know to go update a cpan module.
We have to tell them they don't have the minimum version.
[01:00]
jomoso, as i told - report only the ERRORS - but the above line has no information for me - if the dep is optional, simply it is optional. If it is required, and the neded versions are fullfilled - ok, no error - do not need report - when something has wrong version - report it... basic principe - print only lines what are important for the users...
for the CPAN modules - need to know - install this and this... for the foswiki modules don't need to know anything - the configure should solve the version collisions ;) ;)
[01:05]
gac410Unfortunately "Optional" is only in the opinion of the dev. A lot of MichaelDaum's extensions are very non-functional unless you install all the optional pieces.
Hm. Welll under nytprof, the largest part of installing FilterPlugin is Archive::Tar expanding the tarfile.
[01:07]
jomoyeah - and therefore all those optional extensions shuld be installed by default. - Anyhow - i don't know based ON WHAT i should decide - install the optional or not? - better install everything ;) [01:09]
gac410yeah I don't know. I don't use NatSkin. [01:09]
jomohm...
natskin is easier to adapt to NICER designs and pattern... (imho).
s/and/as/
[01:09]
gac410hm... I was *sure* that foswiki used to try the system untar before using Perl code to expand archives. I don't see that code in 1.2.0 :(
gac410 needs to go digging.
Yeah Someone deleted a bunch of code :P All the code that tries the platform tar / zip is missing.
gac410 was wrong about tar/zip. Code is there. But it prefers perl over system util. Which doesn't make a lot of sense.
[01:12]
okay, well that code is certainly bogus. HUGE speedup by preferring native tar over Archive::Tar. We need to determine if tar present and use system tar if available.
gac410 wonders wtf he was thinking.
[01:23]
jomo:) [01:24]
gac410Wow what a difference FugueIconsContrib. 47 Seconds to install, with NYTProf active. vs. 20 minutes while using Archive::Tar [01:27]
jomoLOL ;) - more than 20 times faster ;) [01:28]
gac410well I guess I should address this for Beta2. Test if "tar" or "unzip/zip" are available, and fallback to perl vs. default to perl. [01:30]
jomohm.. chicken-egg problem. the configure should rescan for the external programs (such grep), when the SafeEnvPath is changed... [01:41]
gac410hm The new configure can have "relationships" between fields. Change A "Check-also" B [01:42]
***ChanServ sets mode: +o Lynnwood__ [01:43]
jomotherefore says the above - it probably has not linked. E.g. my configure finds the "grep" in the /usr/bin and throws a warning about the non-GNU grep. (the OSX's default grep isnt GNU). When i changed the SafeEnvPath to /opt/local/bin:/usr/bin:/bin - e.g. the 1st place is the /opt/local/bin where is my GNU utilities, the configure should change the found grep paths.... [01:46]
gac410I thiink on 1.2, we now just assume grep is on the path and don
don't record the path into the grep command.
[01:47]
jomo(or it is enough stop the warnings - foswiki will probably internally use the SafeEnvPath) [01:47]
gac410gac410 needs to look.
Foswiki internally does use SafeEnvPath.
Oh crap. $Foswiki::cfg{RCS}{initTextCmd} = '/usr/bin/rcs
When CDot refactored into RCSStoreContrib, he went back to a hardcoded path. :(
[01:47]
jomoah - ok - just neded reload the configure after the SafeEnvPath change... ;) - it is a bit configing than the warnings are disappears only after the reload.. ;)
s/configing/confusing/
[01:50]
gac410So RCS Egrep and Fgrep, have the /usr/bin path removed, and just assume they will be found on the path. But the rcs commands are all hardcoded.
That needs a task. We should use same strategy for egrep, fgrep and rcs.
[01:52]
jomoyeah... [01:52]
the lighttpd is not recommended for the "production" use? only for development?
because the tools/lighhtpd.pl is missing from the tarbal - it is only in the trunk...
[01:59]
gac410no idea really, it's only documented as a developers tool. I suppose no real reason that we couldn't include it. But it's never been there. [02:02]
jomook - no problem copy it - only wondering... another - found then all /bin/* commands has changed the shebang to the /usr/bin/env perl - what is great. But the tools/* perl scripts still have hardcoded /usr/bin/perl - it is for reason, or? [02:05]
gac410That's me. We agreed to switch to /usr/bin/env perl for the scripts. It didn't *say* to change the other tools, so I err'd on the side of caution. [02:06]
jomo:) ok [02:07]
gac410Item13323 [02:07]
FoswikiBothttp://foswiki.org/Tasks/Item13323 [ Item13323: Now that Taint mode removed, switch from /usr/bin/perl to /usr/bin/env perl ] [02:07]
***ChanServ sets mode: +o Lynnwood [02:08]
gac410I left the task open, since I commented I was unsure what to do about the rest.
One of the many lose ends
[02:08]
jomo;) [02:12]
.... (idle for 16mn)
GithubBot[distro] gac410 pushed 1 new commit to master: http://git.io/vebEX
distro/master 32f7a68 George Clark: Item13349: Partial fix for installing large extensions...
[02:28]
***GithubBot has left [02:28]
FoswikiBothttp://foswiki.org/Tasks/Item13349 [ Item13349: Installation produces perl code on screen. ] [02:28]
gac410jomo, I'll open a task for the RCS commands having hardcoded path. [02:29]
jomocool, theyre in ./lib/Foswiki/Contrib/RCSStoreContrib/Config.spec and ./lib/Foswiki/Sandbox.pm too [02:32]
gac410Item801: [02:32]
FoswikiBothttp://foswiki.org/Tasks/Item801 [ Item801: Configure needs to more reliably set the grep program path, and handle configuration issues ] [02:32]
gac410I reopened that for rcs.
It's just a comment in Sandbox.pm.
[02:32]
jomoah - i don't see the context - just grepped the sources for /usr/bin [02:34]
gac410yeah.
It's in a =cut block
Damn... a new blocker. "Force a new revision" ... isn't
The url param "forcenewrevision" occurs 3 times in the post, Bleh
[02:34]
jomogtg sleep... 04:40 here... bye for now [02:44]
gac410g'night [02:44]
.... (idle for 19mn)
GithubBot[distro] gac410 pushed 1 new commit to master: http://git.io/vebwN
distro/master dc55210 George Clark: Item801: Don't hardcode rcs program paths
[03:03]
***GithubBot has left [03:03]
FoswikiBothttp://foswiki.org/Tasks/Item801 [ Item801: Configure needs to more reliably set the grep program path, and handle configuration issues ] [03:03]
....................... (idle for 1h50mn)
***gac410 has left [04:53]
............ (idle for 59mn)
ChanServ sets mode: +o CDot [05:52]
ChanServ sets mode: +o MichaelDaum [05:59]
........ (idle for 35mn)
MichaelDaumCDot, did you read http://foswiki.org/Tasks/Item13364 ? [06:34]
CDotCDot has only just got up [06:36]
CDot commented
not a good start to the day, if I'm honest.
[06:45]
.................... (idle for 1h36mn)
GithubBot[distro] MichaelDaum pushed 1 new commit to master: http://git.io/veN6O
distro/master fde394d MichaelDaum: Item13365: proper use of forcenewrevision
[08:21]
***GithubBot has left [08:21]
FoswikiBothttp://foswiki.org/Tasks/Item13365 [ Item13365: forcenewrevision checkbox is not working. ] [08:21]
................................................. (idle for 4h0mn)
MarcKloterHey there. Is there a macro just like %REVINFO% but for attachments? So I can get the attachment time info but for an attachment in another topic than the current one? :) [12:21]
GuilainCMarcKloter, take a look of listattachmentPlugin
you can get a lot of information about the attachment (in topic, other) and render the attachment list (or one attachment) in the manner you wan
wan = want
[12:33]
MarcKloterOkay, thank you [12:36]
jomoGuilainC: mean http://foswiki.org/Extensions/AttachmentListPlugin :) [12:36]
GuilainCexactly jomo ;) [12:37]
.... (idle for 15mn)
***ChanServ sets mode: +o Lynnwood [12:52]
..... (idle for 21mn)
ChanServ sets mode: +o gac410 [13:13]
gac410Hi MichaelDaum thanks for fixing the forcenewrevision. I swear it was working at some time. I use it occasionally during testing. :( [13:18]
MichaelDaumme2....legacy piles up faster than we can chew it away [13:19]
gac410:) [13:19]
CDot: I looked a couple of times at how to hide the "Files copied" reports in the Extension installer. I wanted to put them behind a twisty, but because the output goes to both shell and web targets, that just didn't seem right.
Item11834
[13:26]
FoswikiBothttp://foswiki.org/Tasks/Item11834 [ Item11834: Hide the details in the extension installation report. ] [13:27]
CDotgac410: "#" is your friend..... 3:-) [13:27]
gac410gac410 doesn't follow that. needs more coffee [13:27]
CDot# - the perl comment character [13:29]
gac410Oh... you mean remove the details? :(
gac410 wants them, but really only in the logfile written in the configure/logs directory
Maybe only print details when #ASSERTS are enabled? I've found them useful in debugging what fails
[13:29]
CDotgood idea
CDot uses if (DEBUG) quit a lot for that
[13:31]
gac410Maybe it's gotten more reliable, but when I first did the rfactoring of Package, ... it was pretty fragile. Would be nice if the results could come out to the web "along the way" rather than waiting for the end. Especially when dependencies are installed. [13:33]
CDotsure; but that's an ajax call or ten further than I was prepared to code :-( [13:36]
gac410yeah. That's a long standing issue. [13:36]
.... (idle for 16mn)
CDot .. hope it wasn't something I said :) [13:52]
Lynnwoodjust did first install of 1.2 beta on dreamhost. very nice! [14:05]
gac410great to here Lynnwood
er... hear
[14:08]
Lynnwoodonly little glitch in docs is that in the 5 minutes install notes, on point #2 about configuring apache, it would be worth adding short comment for installing in shared hosting noting that you simply have to (at least) mv bin-htaccess.txt to bin/.htaccess [14:10]
gac410Make it so :) [14:10]
Lynnwoodthen the bootstrap works
y, i'll see to that.
sorry i'm not up on it.... should i be adding notes to BetaTestCoverage?
[14:10]
gac410Sure. I don't think anyone else is yet. [14:12]
Lynnwoodfiling tasks for docs?
Lynnwood needs to get github working
It's not exactly clear how i should use BetaTestCoverage....
I see column for user and "task and notes". Should we make notes right in the column or put links there to notes.
[14:12]
GithubBot[distro] gac410 pushed 1 new commit to master: http://git.io/vexwC
distro/master 8e2f345 George Clark: Item13367: Adjust tml link regex to be less greedy...
[14:14]
***GithubBot has left [14:14]
FoswikiBothttp://foswiki.org/Tasks/Item13367 [ Item13367: The Extension "Compatibility" field is corrupted in the Extensions report in configure ] [14:14]
Lynnwoodsorry if i'm being a bit dense... not in top form today... woke up with spring sinus headache.
The new configure is excellent!
[14:15]
gac410Lynnwood: I'm not really sure how best to go about. I was figuring small stuff could go inline, along with links to tasks. It was really a very quick brainstorm on how to track beta results, with no serious thought given to next steps. [14:15]
Lynnwoodsure
understood.
[14:16]
gac410Lynnwood: I'm with you on Configure. I think it's a huge improvement. [14:16]
Lynnwoodwiki-way... i'll just start making some notes there and we can figure how to organize better as they develoip [14:16]
gac410Docs, use Item9693 [14:16]
FoswikiBothttp://foswiki.org/Tasks/Item9693 [ Item9693: Documentation updates for Foswiki 1.2.0 ] [14:16]
Lynnwoodcool [14:16]
gac410You can actually just edit and commit on GitHub directly and it should work.
Just there is no checker to make sure that the commit message is in Itemxxxx format, or that topic timestamps are fixed. :(
[14:17]
Lynnwoodright. i'll see about getting fully setup with github later
i'm way overdue to get familiar with the workflow of it... :-(
[14:18]
gac410One reminder ... Always do a "git pull --rebase" BEFORE you "git push" That way your fixes are on the top of the heap. [14:19]
Lynnwoodthanks [14:19]
gac410If not, you end up with a big merge commit of everyone elses work. [14:19]
Lynnwoodso configure no longer requires password to save changes?
Guess that works fine once configure is restricted.
it was alway a pia to remember separate configure pw anyway (and browser always wants to use default pw for site)
[14:33]
gac410Yes. correct. Plus using a "shared secret" admin password was considered very poor security practice.
You can still create an admin password, but foswiki works fine without it.
[14:34]
Lynnwoodby "shared secret" you mean that one that more than one person uses? [14:35]
gac410yes [14:35]
Lynnwoody, i get that [14:35]
gac410MichaelDaum: I have a quick js question if you have a moment. In configure.js, line 331, I'm trying to get a linebreak into a text area. I've tried both \n\n, and <br/> \n\n is just ignored, wrapped all togehter. <br/> prints inline rather than being interpreted.
I want to add a message"
$div.text( 'The following Wizards are running, Please wait:\n\n'
+ (params.wizard ? params.wizard + " " : '')
[14:42]
MichaelDaumlet me see [14:43]
gac410Trying to address one of jomo's feedback about meaningless dialog boxes [14:43]
MichaelDaum$div.html("...<br />"+...) [14:44]
gac410Ah... okay. Thanks. I'll play with it. [14:44]
MichaelDaum$div.empty().append("Some Text").append("some more text")
http://jqapi.com/#p=append
spare the empty() when the div was already empty
[14:45]
LynnwoodNice... so new configure simply uses authentication of AdminUsers on site. [14:49]
gac410okay. thanks. that works. $div.html(...) and then $div.append() for the messages. Great. One little issue addressed.
Lynnwood: Yes, OR you can set a list of configure users in Security & Authentication
Some small sites might just want admins to use configure. Others might want to restrict it to "server admins" as opposed to Wiki admins
[14:50]
Lynnwoodright [14:51]
gac410If you list users in Security&Authentication Access Control tab, then AdminGroup membership is ignored. [14:52]
GithubBot[distro] gac410 pushed 1 new commit to master: http://git.io/vexdR
distro/master 5e834c2 George Clark: Item13364: Add a note about active wizards...
[14:55]
***GithubBot has left [14:55]
FoswikiBothttp://foswiki.org/Tasks/Item13364 [ Item13364: Very bad user experience with the new configure - installing extensions ] [14:55]
gac410Thanks for the help MichaelDaum ... I got a message into the wizard box.
gac410 now needs to go work on taxes. :(
[14:57]
.................................... (idle for 2h58mn)
***ChanServ sets mode: +o CDot [17:55]
.... (idle for 15mn)
gac410hey CDot... welcome back. I hope I didn't chase you away :D [18:10]
CDotnah, had to go and pick up a cylinder. Diving on Sunday, got to be prepared!
CDot is just looking at the installation dialogs
[18:11]
gac410gac410 is going to check in all the <Extension>/.gitignore files. Used the build.pl gitignore target to add all the building a release work products. Getting annoyed at big git status :P
gac410 is wondering if CharSetConverterContrib ought to be shipped by default. It's just one utility, and it's now referenced in our Foswiki:System.UpgradeGuide01x02
[18:11]
FoswikiBothttp://foswiki.org/System.UpgradeGuide01x02 [ UpgradeGuide01x02 ] [18:13]
gac410CDot .. that's another area that needs review. I refactored the upgrade guide, moved the TWiki ancient history into Support web, and tried to make sense out of the steps to Convert ACLs, Convert Charset, and Change Store. [18:15]
CDotconfigure first; can't multitask when deep in complex code [18:15]
gac410np. what Configure?? Complex ?? :D
gac410 will update Package.pm and suppress all the "copied file" statements. Maybe add a summary. "Copied n files, checked in n files. or something."
Diving... hopefully someplace with warmer waters. I did all my diving years ago, in cold NewEngland water. Fun, but still pretty miserable at times ;)
[18:16]
.... (idle for 19mn)
CDotno, the water is pretty cold right now; but we dive in dry suits, so not a problem until it gets below ~4 degrees
actually the extensions installer wizard is surprisingly simple. I just never went "all the way" before (I was pretty exhausted after the configure recycling)
[18:37]
.... (idle for 17mn)
gac410Dry suits, that's nice. Though iirc our dive instructors were pretty cautious about buoyancy control in them. I never tried one. [18:55]
CDotit takes practice, but it's worth it in the cold water (and we're generally warmer than New England, if I remember my north atlantic isothermic charts correctly) [18:57]
gac410It was 30+ years ago for me, so I don't remember all that much ... other than it was cold. Though we had one dive buddy that took forever and a day to get into her suit, so we'd sit in the water sweltering while she struggled. [19:00]
GithubBot[distro] gac410 pushed 1 new commit to master: http://git.io/vehNw
distro/master 52433f8 George Clark: Item13135: add/update Extension & core .gitignore files...
[19:08]
***GithubBot has left [19:08]
FoswikiBothttp://foswiki.org/Tasks/Item13135 [ Item13135: .gitignore management is unclear ] [19:08]
........ (idle for 36mn)
gac410Hey CDot. pita rm here. See Item13352 and http://trunk.foswiki.org/Sandbox/TestItem13352 I recreated it. Indeed Table handling crashes if an extraneous </verbatim> tag appears in the topic. [19:44]
FoswikiBothttp://foswiki.org/Tasks/Item13352 [ Item13352: EDITTABLE macro is ignored if color macros are present in same topic (30th March 2015 version). ] [19:44]
gac410Looks like $this->{waiting} is undefined.
Adding an extraneous </verbatim> into the Table Parser tests doesn't crash :(
Hi JulianLevens I checked in a whole mess of .gitignore updates. Added all the "output files" from building extensions or the foswiki release
They are never pseudo-installed, so they need to be in the individual extensinos.
[19:49]
JulianLevensI noticed that and that it included TinyMCEPlugin and I noticed extra lines like /#pub/System/TinyMCEPlugin/... which looks odd to me [19:56]
gac410CDot: I have a failing unit test for Item13352. I'll check it in
JulianLevens: oh. damn. I missed deleting those. Those are comments in the MANIFEST. I deleted them when I saw them. I suppose I ought to fix the "gitignore" target of BuildContrib
[19:57]
GithubBot[distro] cdot pushed 1 new commit to master: http://git.io/vejcp
distro/master 74a7bc1 Comment: Item13364: refactored to support bulk operations on extensions. Pretty much untested, so help desperately needed!
[19:59]
***GithubBot has left [19:59]
FoswikiBothttp://foswiki.org/Tasks/Item13364 [ Item13364: Very bad user experience with the new configure - installing extensions ] [19:59]
JulianLevensRef: Convert ACLs, Convert Charset, and Change Store: I reckon they should be combined into one script [19:59]
CDotgood grief no; they do utterly different things [20:00]
gac410They work very differently. ACL and Charset are "in place" Change store makes a copy, and does not copy some critical files. [20:00]
CDotok george, if you encounter any problems with that configure fix (that you can't fix straight away) just append them to Item13364 [20:02]
JulianLevensMaybe, I think I added the charset converter code to change_store, it makes sense to convert as you copy [20:02]
GithubBot[distro] gac410 pushed 1 new commit to master: http://git.io/vejls
distro/master 2d1a0ea George Clark: Item13352: Failing unit test...
[20:02]
***GithubBot has left [20:02]
CDotI will have maybe an hour tomorrow to look at them [20:02]
gac410Okay CDot ... I'll test it and see how it goes. [20:02]
CDotta. g'night! [20:02]
gac410g'night and thanks!
JulianLevens: just be careful, as the change_store does not copy subdirectories of pub. So System extensions that put their images, javascript, etc. in subdirs, will be damaged if you convert "System"
Also, charset converter is intended to run on 1.1.x as well. It's not just a 1.2 thing.
[20:02]
JulianLevensYes, lack of pub subdir copies is a store api limittaion [20:05]
gac410Where as the change store is *only* 1.2, and the ACL converter is ... well, a hybrid. I'm supposed to backport the fixes into a 1.1.9 patch contrib.
Or at least that was the original intention
[20:05]
JulianLevensI did start work on a generic store.pl with support for many options provided by extra .pm files extending the capabilities. Thus allowing a lot of code sharing [20:08]
gac410cool [20:08]
JulianLevensAlas, it needs some hacking, I was also experimenting with a feature proposal at the time (changes to core and the store api) so I cannot submit it in it's present form [20:10]
gac410JulianLevens: I've fixed BuildContrib target gitignore, won't pick up comments in MANIFEST files. I'll check that in along with the TinyMCE .gitignore file. [20:11]
JulianLevensExcellent [20:13]
GithubBot[distro] gac410 pushed 1 new commit to master: http://git.io/vejEP
distro/master 2f2901f George Clark: Item13135: Don't gen ignore lines for comments...
[20:13]
***GithubBot has left [20:13]
FoswikiBothttp://foswiki.org/Tasks/Item13135 [ Item13135: .gitignore management is unclear ] [20:13]
JulianLevensWhere is the convert ACL script? [20:15]
gac410tools/convertTopicSettings.pl
It can do a lot more than just the ACL changes.
[20:16]
LynnwoodI have a rather un-orthodox (and possibly bad/mad) idea that i'd welcome feedback on. A client has some static html documentation that's generated by generated by an external application (JsDoc Toolkit) and we're exploring option for integrating it into documentation created within Foswiki.
I explored creating some special template files for JsDoc Toolkt such that it would create html files with only the body content (no html, header, body tags etc) and with "txt" suffix rather than "html" and then just dropping them into a web directory.
The first part of that was doable, but surprisingly, it did not seem possible to specify different file suffix.
[20:21]
gac410You might be able to pass them through the WysiwygPlugin HTML2TML converter. [20:25]
Lynnwoodhmmm. i wasn't really that worried about html within the body since foswiki would deal with that ok. [20:25]
JulianLevensWhy could you not specify txt suffix? Did JsDoc complain? [20:26]
LynnwoodJulianLevens - that what i figured initially, but there actually is no option to specify suffix.
".html"
is baked in
[20:26]
JulianLevensCould you not drop as html and then another to rename all the files?
another script
[20:27]
Lynnwoodpossibly...
Perhaps a bigger problem was that JsDoc insisted on creating a couple levels of sub-directories
[20:28]
JulianLevensThere's probably a line command to rename them all [20:28]
Lynnwoodand includes links to those. [20:28]
JulianLevensWhat's in the sub-directories? [20:28]
LynnwoodJulianLevens - the other issue about the file suffixes is that internal links are created with the .html suffix.
additional content.
also assets such as css files, but i could deal with that within foswiki.
(i.e. i would create custom template files for this content that pulls in those assets)
maybe... assuming foswiki would allow that if they weren't in pub directory... probably not.
[20:29]
JulianLevensIt sounds like it's possible, but you'll need to write a script to do some conversion and renaming s [20:31]
Lynnwoodyea... probably.
anythings possible with enough extra coding
[20:31]
GithubBot[distro] gac410 pushed 1 new commit to master: http://git.io/vejoR
distro/master ddb0c17 George Clark: Item9693: &mdash; was saved as UTF8
[20:32]
***GithubBot has left [20:32]
FoswikiBothttp://foswiki.org/Tasks/Item9693 [ Item9693: Documentation updates for Foswiki 1.2.0 ] [20:32]
JulianLevensBased on what you've said, it sounds simple enough but ... [20:33]
LynnwoodI'm guessing that foswiki would balk at serving up html files that were in folder under data directory...
JulianLevens - you're encouraging me to consider further.
maybe such a script would not be too difficult.
[20:33]
JulianLevensYour script may also need to rename data/ references to pub/ as well as .html to .txt
I would think a first stab would only take an hour or so (typical programmer optimism)
[20:36]
Lynnwoodit would also need to rewrite links in all the files
and possibly squash directory hierarchy
but doable...
[20:37]
..... (idle for 23mn)
***JulianLevens has left [21:01]
............. (idle for 1h1mn)
GithubBot[distro] gac410 pushed 1 new commit to master: http://git.io/vve3a
distro/master 3fceacb George Clark: Item11834: Item13364: Suppress noise when copying files...
[22:02]
***GithubBot has left [22:02]
FoswikiBothttp://foswiki.org/Tasks/Item11834 [ Item11834: Hide the details in the extension installation report. ] http://foswiki.org/Tasks/Item13364 [ Item13364: Very bad user experience with the new configure - installing extensions ] [22:02]
gac410hey jomo, with that checkin,. most of the noise is gone from installing extensions, unless you enable ASSERTs in LocalLib.cfg
And CDot addressed the "one at a time" issue.
[22:03]
jomowow... will check it tomorrow - that's a speed.. :) - will try again install NatSkin :) [22:04]
GithubBot[distro] gac410 pushed 1 new commit to master: http://git.io/vveGg
distro/master 2130a71 George Clark: Item13349: Adjust test for changed error msg
[22:08]
***GithubBot has left [22:08]
FoswikiBothttp://foswiki.org/Tasks/Item13349 [ Item13349: Installation produces perl code on screen. ] [22:08]
gac410Well some of the bugs are not fixed. :(
Just tried to install ControlWikiWordPlugin, The "undef" error came back.
And the specs were messed up.
I have not updated the beta. So you'll need to test with github checkout.
[22:17]
jomoofc, no problem - will test it and report errors - as you said - NatSkin = ACID TEST :) :) [22:18]

↑back Search ←Prev date Next date→ Show only urls(Click on time to select a line by its url)