#foswiki 2016-01-05,Tue

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

WhoWhatWhen
***GuilainC_away is now known as GuilainC [05:42]
.... (idle for 15mn)
GithubBot[distro] gac410 pushed 2 new commits to master: http://git.io/vulVO
distro/master 93bba3a George Clark: Item13912: Add =pod marker...
distro/master daf3a04 George Clark: Item13912: Configure should use pod as well...
[05:57]
***GithubBot has left [05:57]
FoswikiBothttp://foswiki.org/Tasks/Item13912 [ Item13912: perldoc does not process bulk_copy.pl correctly because of missing pod marker ] [05:57]
***gac410 has left [06:04]
............ (idle for 58mn)
ChanServ sets mode: +o MichaelDaum [07:02]
GuilainC is now known as GuilainC_away [07:14]
................ (idle for 1h19mn)
ChanServ sets mode: +o CDot [08:33]
.................................................. (idle for 4h7mn)
SvenDowideit_hum
when I goto http://foswiki.org/Main/SvenDowideit
I get
No permission to view WikiDashboard
[12:40]
oh, that's probably because i disabled it to reduce server load? [12:52]
.... (idle for 17mn)
***ChanServ sets mode: +o CDot [13:09]
MichaelDaumSvenDowideit_, your homepage takes half a minute to load
vs Main.WebHome loading in 600ms from over here
[13:13]
***ChanServ sets mode: +o gac410 [13:21]
JulianLevensga410: we're meeting on #foswiki-association in half an hour, can you join us? [13:29]
gac410I'll be there [13:29]
Hi CDot .. Do you have a minute? [13:35]
CDotgac410: aye, as long as it doesn't involve too much thinking ;-) [13:42]
gac410Hopefully not too much. I made a rather innocuous change in Oct. setlib.cfg will report the 7 critical dependencies that break foswiki so badly that we can't put up an error.
But just discovered that it breaks the CLI utilities on windows. Doing a "require CGI" to see if it is present, sets binmode on STDIN, breaking the prompting.
because chomp no longer chomps
[13:43]
CDotand you can't local STDIN because.... ? [13:45]
gac410Ah. I didn't try that. I added "binmode( STDIN, ':crlf') [13:45]
CDotI don't know if local works on globs, but that's the first thing I'd try [13:46]
gac410So add a "local STDIN" to the tools/scripts? or add it to setlib where it probes for CGI? [13:46]
CDotin a block
{ local STDIN; require CGI; ... }
[13:46]
gac410Ah okay. In setlib.cfg . Okay I'll test that out. Thanks! [13:47]
***ChanServ sets mode: +o Lynnwood [13:47]
CDotit probably won't work, from reading.
I'd guess you probably need to reopen STDIN
[13:48]
gac410the "binmode( STDIN, ':crlf')" did work. But annoying that it has to be done on each script that requires setlib.cfg [13:49]
CDothttp://docstore.mik.ua/orelly/perl/cookbook/ch07_17.htm [13:49]
***Lynnwood has left "Textual IRC Client: www.textualapp.com" [13:51]
gac410I don't really understand why a "require CGI" would cause it to execute to the point of issuing the ->binmode() commands. There is no BEGIN block, but clearly it was the cause. I dropped the require for CGI and the scripts started working.
But answering that probably borders on requiring too much thinking :D
What sent me down this rathole is I added the setlib ... exactly like tools/configure to tools/rewriteshebang to remove the cryptic need for -I ../lib
no good deed goes unpunished
[13:51]
...... (idle for 26mn)
ping gmc. Are you around? [14:20]
............. (idle for 1h2mn)
andreliHello. I hope you all started well into 2016. All the best to you all and 2016 releases of Foswiki in particular. [15:22]
gac410Thanks andreli
You too!
[15:22]
andreliThank you gac410
A quick question: As I usually only hack some lines here and there, I have poor knowledge of all the Foswiki APIs.
How would I best log within Foswiki - the 'propper Foswiki way'
I would like to add a custom logging of special User Mapping Events
[15:23]
gac410I'm pretty sure that we escalated the logger to part of the official foswiki API.
See Foswiki::Func::writeEvent()
[15:25]
FoswikiBothttp://trunk.foswiki.org/System/PerlDoc?module=Foswiki::Func [15:26]
andreliOk, I see events, errors and debug, but how do I get a dedicated file (log), let's say mapping.log? [15:29]
gac410Ah... hm I misunderstood your question. We don't really have a supported way for that
The logger is designed around the "standard" log4Perl / log4J log levels. warning, fault, debug, info ,./.. iirc there are 7 of them.
The (never released) LogDispatchContrib added a lot more flexibility, but all that work stopped when the dependencies proved to be too complex.
[15:29]
andreliAs I understood log4..., we use log4php in all our apps, the different log levels go into the same log file. You can setup easily different files to log into. But for the time being, I will work with writeDebug(..).
Thanks
[15:34]
gac410LogDispatch can flexibly direct a log level into separate files.
andreli: iirc, Foswiki::Logger is now considered public API, so you have a bit more flexibility. but the Compat and PlainFile loggers can't specify what file a level filters into.
[15:36]
FoswikiBothttp://trunk.foswiki.org/System/PerlDoc?module=Foswiki::Logger [15:38]
andreliAn other, more basic Perl question. I see a lot a noise in my Apache error Logs:
return 1 if $cUID eq $this->{session}->{user};
Use of uninitialized value in string
How do I get this line quiet: lib/Foswiki/Users/TopicUserMapping.pm line 574
[15:40]
gac410What's undefined? The session user, or the cUID
Something strange is going on, I don't ever see that warning. Are you doing something unusual with your mapper / login manager, / ...
[15:42]
andreliI see the warning, when I get a login for a user that I don't have a mapping for.
The edge case might be, that I authenticate users, LdapApacheLogin, that I do not provide a mapping for, LdapUserMapping.
Example: Shared accounts can provide login credentials, but do not map to a wiki user.
[15:44]
CDotgac410: I've done a correct implementation of TMPL (with unit tests). Any preference how to check it in (or do you want me to attach a patch?)
(the EOTC_ patch is badly SNAFU, by the way)
[15:47]
MichaelDaumCDot, did you see my 2c on f.o about it? [15:49]
CDotMichaelDaum: before or after mine?
i.e. I saw your 2c, but not the full dime.
[15:49]
MichaelDaumhttp://foswiki.org/Tasks/Item13909#r10 [15:50]
CDotgotcha; we're on the same page. [15:51]
MichaelDaumokay
wrt checking in: have a feature branch
[15:51]
CDotWith my TMPL: implementation you can write (for example) %TMPL:QUERYSTRING% to get the full query string from the save
(not that you'd ever want to)
[15:52]
MichaelDaumnot sure what your code is about to impl [15:52]
CDotjust making the point that TMPL: allows all macros to be used during tmplate topic expansion *without* special handling
sorry, will attach a patch to the fetaure proposal
[15:52]
MichaelDaumor use an item branch [15:53]
CDotargh, git..... [15:53]
gac410item/feature branch is very easy. git checkout -b Item13909 creates a new local branch. Commit your changes, then git push origin Item13090 [16:06]
FoswikiBothttp://foswiki.org/Tasks/Item13909 [ Item13909: Support TWiki %EOTC_ prefix to expand individual wiki MACROS during topic creation from template. ] http://foswiki.org/Tasks/Item13090 [ Item13090: extensions tab needs to be rendered differently vs the rest of the tabs ] [16:06]
gac410hm CDot. so the requirement is, the ability to expand a *single macro* within a template when it expands on topic creatiion. [16:07]
CDotright [16:07]
gac410EOTC seems that it would do that just fine. Any macro prefixed with EOTC__ would expand. [16:07]
CDotyes, but the expansion is clumsy, and done in the wrong place [16:08]
gac410%EOTC__IF{}% is matched as a macro, When encountered during create, the EOTC__ is stripped, and %IF{}% is expanded.
Clumsy, yes indeed, an awful implementation.
[16:08]
MichaelDaum%EOTC_TMPL:P{"foo"}% does not work [16:09]
gac410No... this works for TML macros TMPL is not TML. [16:10]
MichaelDaum;) it is ... sometimes [16:10]
CDotMichaelDaum: no, and TMPL:P in a template topic won't work either
at least, not currently. Though that could be added, I guess.
[16:10]
MichaelDaum%TMPL:P{"hello" \n param="foobar"\n"}% _is_ expanded during macro expansion i.e. not during tmpl [16:11]
CDotis it? OK, then it will work in a template topic [16:11]
MichaelDaumas well as $percntTMPL:P{\"yadda\"}$percnt [16:11]
gac410And in that case I'm totally baffled. [16:11]
CDotthough you may have to write %TMPL:TMPL:P
gac410: why?
[16:11]
gac410my understanding of templates is tenuous at best, and this is killing me. [16:12]
MichaelDaumnot last time I clicked on my fw [16:12]
CDotsee patch and description in http://foswiki.org/Development/SupportAllMacrosInTemplateTopics
MichaelDaum: I mean after my patch is applied :-)
[16:12]
MichaelDaumsee Foswiki.pm ... has got a line 335 [16:12]
gac410Anyway, I don't see the connection. User asked for way to expand %IF{}% "during topic creation" ... How on earth does %TMPL:QUERYSTRING% relate to that. [16:13]
CDotMichaelDaum: oh, barf. That's nasty. And yes, I probably added it :-(
gac410: because %TMPL:IF{}% expands during template instantiation time too
and so does %TMPL:ANYOTHERMACRO{...}%
[16:13]
gac410Ah... okay. so %TMPL:anymacro{}% gotcha. nice. [16:14]
CDotthe patch is in fact one line; the rest is test refactoring and some missing /o's [16:15]
gac410So we are in feature freeze, and I was about to do a final task review, address the STDIN on windows snafu I caused, Do you feel we should pull this into 2.1?
And then build 2.1 beta and announce to the translators.
[16:15]
CDotDeffo not for a bugfix patch. Clearly a new feature.
It has unit tests and docco.
[16:16]
gac410Right, so it's a new feature and 2.1 is a feature branch. The question is jumping the "feature freeze" we entered just before christmas. [16:17]
CDotup to you. [16:17]
gac410That is what I was afraid you'd say :P [16:17]
CDotI wouldn't consider it to be an important feature, myself [16:17]
MichaelDaumMichaelDaum loves one-line patches [16:17]
CDotbut it *is* a feature [16:17]
gac410yeah I agree.
Can you push to an item branch?
CDot, for 2.1 and 2.0.4, there is also a one-liner bugfix in Item13906.
[16:18]
FoswikiBothttp://foswiki.org/Tasks/Item13906 [ Item13906: EditRowPlugin create table row without spaces. ] [16:19]
gac410Given it is deep in table code I am very reluctant to just commit it. [16:20]
MichaelDaumlet's please leave it some time to peer-test it [16:20]
CDotif the unit tests pass, I can hardly protest :-) [16:20]
MichaelDaumlast but not least announce it on a feature proposal [16:21]
CDotMichaelDaum: what, like http://foswiki.org/Development/SupportAllMacrosInTemplateTopics? ;-) [16:21]
gac410Well they pass with a change. But it does seem to be rather innocuous, and the tables "look right" [16:21]
CDotthe patch looks correct [16:22]
gac410Okay. RM decision. I'm going to not pull it into 2.1. Last minute features tend to come back and bite us. [16:22]
MichaelDaumCDot, there you are :) [16:22]
gac410Thanks CDot. I'll apply that one then.
We'll put it on the list for 2.2, and will merge it into master once 2.1 ships.
MichaelDaum: The JQueryAjaxHelper bug building the view URL ... can you address that?
Item13898
[16:22]
FoswikiBothttp://foswiki.org/Tasks/Item13898 [ Item13898: JQueryAjaxHelper =jumpbox= code does not work with short URLs ] [16:24]
MichaelDaumI'd address it by removing the jumpbox example [16:24]
gac410:P And I was going to USE the jumpbox example, which is why I found the bug [16:24]
MichaelDaumit isn't used anywere
heh >:]
[16:24]
gac410Well actually I wanted to use it as the autocomplete for topic parents
Pick from a dropdown list is really awful. Autocomplete would be much nicer.
But autocomplete wouldn't work because I use short urls.
[16:24]
CDot, the big advantages of using a feature / item branch on github: 1) Your RM can just say "git checkout master; git merge Item12345" and poof ... feature is incorporated.
and 2) You get all the attribution for the commits. If I cut/paste a patch, it shows up under my authorship.
and 3) the commits all get the same ID. So "git branch --contains <somecommithash>" shows you where the commits ended up. Cherry pick and cut/paste don't do that.
[16:35]
FoswikiBothttp://foswiki.org/Tasks/Item12345 [ Item12345: upgrade jquery and other 3rd party plugins ] [16:35]
..... (idle for 20mn)
gac410Okay CDot ... I'm confused. I thought we decided to NOT use /o on regexes, as the recent guidance is that it does nothing and introduces bugs.
And your proposed patch adds them back in.
[16:55]
MichaelDaum: I'v got someone asking for single-quote to be treated as a TML markup delimiter. Ie Wants to write =widget='s where widget is fixed font.
our $ENDWW   = qr/$|(?=[\s,.;:!?)])/m; Simple enough - one-character one-liner. Add a ' to this regex. But I suspect touching this area of render would be very very dangerous.
fingers stuck in wall socket dangerous ... while standing in water dangerous, ;)
[17:06]
...... (idle for 27mn)
***ChanServ sets mode: +o Lynnwood [17:34]
........................................ (idle for 3h16mn)
GuilainC_away is now known as GuilainC [20:50]
.... (idle for 16mn)
GithubBot[distro] gac410 deleted Item13405 at 2fdc6c1: http://git.io/vu00t [21:06]
***GithubBot has left [21:06]
FoswikiBothttp://foswiki.org/Tasks/Item13405 [ Item13405: Unicode Normalisation ] [21:06]
GithubBot[distro] gac410 pushed 2 new commits to Release02x00: http://git.io/vu0E8
distro/Release02x00 9df91eb George Clark: Item13912: Add =pod marker...
distro/Release02x00 318de0e George Clark: Item13912: Configure should use pod as well...
[21:10]
***GithubBot has left [21:10]
FoswikiBothttp://foswiki.org/Tasks/Item13912 [ Item13912: perldoc does not process bulk_copy.pl correctly because of missing pod marker ] [21:10]
.... (idle for 16mn)
foswiki_irc9Hello all, I have an issue to UserInfoPlugin: Lastvisitors. How is it possible, that Admin and UnknowUser are not display any more? [21:26]
gac410What version of foswiki? [21:27]
foswiki_irc91.1.0
1.1.10
Yesterday, I have told you, I have updated from 1.1.4 to 1.1.9. Today I have updated to 1.1.10 :)
remember?
I got problems with perl 5.22
[21:28]
gac410foswiki_irc userids are anonymous - [21:29]
foswiki_irc9okay [21:30]
gac410It's just a rotation I think ... anyway, when did the userinfo stop working. on 1.1.9 or only after 1.1.0
er 1.1.10
[21:30]
foswiki_irc91.1.4
now, I see all users AND admin+unknow user
this is not so nice
[21:30]
gac410From a quick look at the code, it definitely won't work on Foswiki 2.0. It reads the session files directly and Foswiki 2.0.2 changed the format.
You'd be best to have asked around 8 hours ago. MichaelDaum is that plugin's author and he is in Europe.
[21:31]
foswiki_irc9okay, so it is better to ask in a few hours... [21:33]
gac410I'm not aware of any changes with Foswiki 1.1.x that would have broken that extension.
I don't really understand what you mean by Users AND admin+unknown ... but I'm really not familiar with that plugin.
[21:33]
foswiki_irc9It means, that I can see me and other Users plus the Admin User and Unknow User. I think the last one is, because when you try to login and wiki doesn't know you at this moment
thanks a lot. I will come in 10-16hours
have a nice day
[21:35]
gac410You too. [21:39]
..... (idle for 23mn)
GithubBot[distro] gac410 pushed 2 new commits to master: http://git.io/vu09q
distro/master 89cb46c George Clark: Item13913: restore chomp behavior on windows...
distro/master 65919f2 George Clark: Item13914: Improve rewriteshebang to locate libpath...
[22:02]
***GithubBot has left [22:02]
FoswikiBothttp://foswiki.org/Tasks/Item13913 [ Item13913: CLI scripts broken on windows. setlib.cfg issues "require CGI" which sets binmode on STDIN and breaks prompts. ]
http://foswiki.org/Tasks/Item13914 [ Item13914: rewriteshebang should use setlib, don't make user enter -I ../lib ]
[22:02]
GithubBot[distro] gac410 pushed 2 new commits to Release02x00: http://git.io/vu090
distro/Release02x00 ed9fb9d George Clark: Item13913: restore chomp behavior on windows...
distro/Release02x00 d8dbb67 George Clark: Item13914: Improve rewriteshebang to locate libpath...
[22:03]
***GithubBot has left [22:03]
............. (idle for 1h0mn)
dldl-workis there a way to set the cell border color for a single table using %TABLE% Macro (or some other method) ver1.1.9
cause it's seeming like I have to gie it an id, then go and change my css manually, which seems like a lot of work for something basic
[23:03]
***GuilainC is now known as GuilainC_away [23:12]

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