ReleaseMeeting01x02_20140721

Details

1. Tasks review - tasks with commits

Continues from ReleaseMeeting01x02_20140616. Green shows current status

  • 12475 (Closed): Move lib/CPAN/lib modules to a separate CpanContrib; George seems to be working on this one.
    • CDot thought that HTML::Parser didn't require an XS module. Discovered it's a new dependency added in a recent version.
    • ACTION gac410 needs to investigate if CpanContrib could provide the backlevel version or HTML::Parser
    • Net::SSLeay will not be possible to include due to external requirements.
    • Change to waiting for release There is no resolution to the HTML::Parser. Going way back the old versions of the parser still use c code.

Tasks waiting for feedback with commits:

  • 12019 (Closed): JQuery fixes mixed in with some non-default extension work. MichaelDaum will resolve
  • 10484 (Closed): Search formfield issues, waiting on Sven & pharvey, comments about needing CDot input. CDot to investigate documenting the restrictions
  • 10203 (Closed): Mostly complete, Needs help with completing ajax MichaelDaum and gac410 will test registration changes
  • 10376 (Closed): Experimental code Foswiki::Address Do we ship this? Discussed offline with pharvey. This is complete and can be closed.
  • 11671 (Closed): Needs review, work maybe complete. Feature discussion.

2. Tasks Review - Release Blockers

Current list: MinorReleaseBlocker - review of the "Confirmed" tasks

  • 12261 (Closed): Someone needs to update docs for the new PageCache.
  • 12477 (Closed): Spurious .changes entries. GeorgeClark will resolve
  • 12705 (Closed):Need help on this. CDot's patch resolves the corruption but causes   to be lost.
  • 12855 (Closed): This can be closed any time. Left open to remind that we should do a final audit before building release.
  • 12887 (No Action Required): Configure loop on Perl 5.8.8. JanKrueger update the Template parser, and on perl 5.8.8, a while loop over a regex gets "stuck" at the position of the first match. Needs to either be rewritten or someone with deep perl knowledge to fix the regex loop.
  • 12925 (Closed): Page cache unit tests were all disabled in rewrite. StephanOsthold offered to help but has limited time over next few weeks.
  • 12926 (Closed): Subscribe links are broken. GeorgeClark will change SubscribePlugin rest handler to permit GET urls.
  • 12931 (Closed): CommentPlugin on trunk issues Need help with this one... CrawfordCurrie ?
  • 12937 (Closed): Perl issues on 5.20. Close. Can reopen if any more issues crop up.
  • 12958 (Closed): HTML label issue with editor. CrawfordCurrie proposed a patch. Can we apply this? Does it need unit tests?

3. Tasks Review - Normal and Low priority

For current list, see Tasks.TasksByRelease

4. Low hanging enhancements?

  • 10824 (Closed): Support for HEAD request, Should it be happen before cache processing?

5. Other news

  • If we make progress on github restructure (PaulHarvey) then we'll freeze SVN at the next release meeting 8/4 and begin migration to git
  • Next meeting - Monday August 4th, 1300Z

IRC Log

  Conversation with #foswiki-release at Mon 21 Jul 2014 08:22:53 AM EDT on gac410@chat.freenode.net (irc)

   (09:00:57 AM) gac410: Hi Stephan, Marcus ... light turnout today
   (09:01:12 AM) ModAcOst: Gues so^^
   (09:07:50 AM) gac410: Hi Andre, Julian ... Lets give it just a couple of more minutes. Unfortunately I don't have a lot of progress to report.
   (09:09:08 AM) gac410: Agenda is here http://foswiki.org/Development/ReleaseMeeting01x02_20140721
   (09:11:36 AM) gac410: 10 minutes past. Lets get started.
   (09:13:27 AM) gac410: For agenda item 1, http://foswiki.org/Tasks/Item12475 I think the bottom line is that there is no version of HTML::Tree that doesn't require a system dependent compile. I went way back in old cpan versions.
   (09:14:18 AM) gac410: So for sites that have to use Foswiki provided CPAN dependencies, they will not have wysiwyg available
   (09:15:12 AM) gac410: i dont' think it's worth trying to build a "cpan installer" that uses cpanm or whatever to locally compile. Sites should be able to install their own dependencies.
   (09:15:30 AM) gac410: Trying to be everything for everyone is just too complex.
   (09:16:00 AM) ModAcOst: I think itʼs OK to ask for a minimal effort for wysiwyg
   (09:16:18 AM) ModAcOst: HTML::Tree should be available amost everywhere
   (09:16:51 AM) andreli: Isnʼt save to assume, that Perl needs this days eleveted rights on a server anyway, no?
   (09:17:08 AM) gac410: The main reason for the CpanContrib is for sites that cannot install cpan modules due to local restrictions I think those are pretty few.
   (09:18:25 AM) gac410: Anyway unless someone figures out something clever, HTML::Tree requires a compile due to c code components. I don't know how to get around it.
   (09:19:59 AM) gac410: The other question is how do we build the distribution. I assume *without* CpanContrib, but we release a 2nd download that can install the contrib. I think that can be deferred until we are closer to releasing.
   (09:20:33 AM) gac410: andreli: I didn't understand your question, sorry                                                                                                                                                    
   (09:21:45 AM) JulianLevens1: As HTML::Parse has been a FW requirement for years, what actually changes?
   (09:22:23 AM) gac410: nothing. Since we included it in lib/CPAN/lib, I suspect that it was incomplete / broken in prior releases,
   (09:22:28 AM) andreli: gac410: Non question. I just assume depending on Cpan in today world of Perl should be fine.
   (09:22:39 AM) gac410: yes. I agree thanks.
   (09:24:01 AM) gac410: Okay. so I'll close the Cpan task. Regarding the other items, Only http://foswiki.org/Tasks/Item10376 had any progress that I'm aware of. I need to set task waiting for release. I discussed it with pharvey the other day.
   (09:26:20 AM) gac410: So moving on to the next item. The MinorReleaseBlocker tasks. ...
   (09:28:12 AM) gac410: http://foswiki.org/Tasks/Item12524 ModAcOst you had some comments on it.
   (09:28:47 AM) ModAcOst: Yes
   (09:29:11 AM) ModAcOst: So basically I have a solution that would work for the request
   (09:29:28 AM) ModAcOst: But it changes the template where the compare page is defined
   (09:29:41 AM) ModAcOst: So any customizations are void
   (09:30:01 AM) ModAcOst: Which is why I wouldnʼt put it into a minor release
   (09:31:02 AM) ModAcOst: However if itʼs not important to any of you I can release it anyways
   (09:31:30 AM) gac410: Hm... Well our minor releases are rather major :) I don't see it as a blocker though. It was MichaelDaum's task, so if he still considers it a blocker, I think it's okay to fix.
   (09:32:01 AM) gac410: If you do, be sure to put a warning in the release notes.
   (09:32:31 AM) ModAcOst: Okay, Iʼll speak with MichaelDaum and will release or postpone
   (09:32:38 AM) gac410: Lets ask Michael to make a decision, Blocker, or defer to 2.0 which could be a long ways away :)
   (09:32:47 AM) gac410: sounds good.
   (09:34:06 AM) gac410: Item12965 - cache deadlocks. That's being worked on, not fully understood yet,
   (09:34:41 AM) JulianLevens1: Can only wait for MD on that
   (09:35:27 AM) gac410: Item12868 is a proposed patch from someone doing a Twiki migration. I suspect that also needs MD and CC review, they had changes related to formfields as well.
   (09:36:36 AM) gac410: Because someone went to work of generating a patch, I'd like to see us at least review / consider it.
   (09:37:37 AM) gac410: Item12261 - documentation for the new page cache. needs someone to work with MichaelDaum
   (09:41:07 AM) gac410: Item12477 - I'm working on that one. I've posted some ideas to the task. I'll probably try to patch it this week. Unfortunately the .changes file was broken in 1.1.x,
   (09:43:02 AM) gac410: Item12705 really need MD and CC to resolve. MD proposed a hotfix, it breaks lots of unit tests. CC has a proposed patch, it causes Wysiwyg to loose all  s in verbatim / literal blocks.
   (09:44:52 AM) gac410: I think Item12855: Audit core DEPENDENCIES, can be a last minute review before building a release. Left it as a blocker so it doesn't get overlooked.
   (09:46:42 AM) gac410: Item12887 configure loops. That's a bug in Perl 5.8.8 jast rewrote some of the configure template parser that exposed it. I can recreate it, but the code is rather "difficult"
   (09:47:52 AM) gac410: Basically perl 5.8.8's current position in the regex parser gets "stuck" and it loops processing the same match over and over. crashed my laptop the first time I hit it.
   (09:53:44 AM) gac410: Item12925 Anyone energetic enough to tackle unit tests? Michael's new cache disabled all the existing tests. Someone needs to rewrite.
   (09:54:25 AM) ModAcOst: Iʼll be willing to help, but next few weeks are quite full
   (09:55:42 AM) gac410: Sold! :D
   (09:55:42 AM) gac410: Item12926: needs a javascript wizard, or we have to change SubscribePlugin rest handler to accept "GET" requests.
   (09:57:07 AM) gac410: the issue with 12926 is that the subscribe links in the menu bar are simple <a href> links. Either javascript needs to convert them to a form post, and somehow apply a strikeone key, or the path of least resistance is to just allow GET urls
   (09:58:12 AM) gac410: That one crops up because 1.2 has made REST secure by default. Restricted to POST, Requires STRIKEONE and must be logged in.
   (09:58:31 AM) gac410: Any of which can be disabled when the rest handler is registered.
   (09:58:59 AM) ModAcOst: The problem with strikeone is that there is no api to load it if there is no form on the page
   (09:59:24 AM) ModAcOst: So if you have a page without a form there will be no strikeone scripts
   (09:59:53 AM) ModAcOst: thus no magic with <a href>
   (10:00:17 AM) gac410: hm Then I guess the solution for 1.2 has to be just to allow get for subscribe. Simple enough, just set a flag in the register rest call.
   (10:00:26 AM) gac410: I'll take that one then
   (10:00:43 AM) gac410: I can manage a one-liner :)
   (10:00:52 AM) ModAcOst: ^^
   (10:01:27 AM) gac410: Item12931 That's crawfords comment plugin changes. Not sure how to resolve.
   (10:02:51 AM) gac410: He says it's a performance tradeoff, I really don't like the way it works currently. Posting comments to tasks on trunk is very confusing. The comment appears truncated.
   (10:04:15 AM) gac410: Item12937 - Perl 5.20 issues - is fixed, except for a few unit tests hash ordering issues that we can ignore. I left it open just in case anything further breaks, but I can close it.
   (10:05:29 AM) gac410: And the last blocker in Confirmed state Item12958 has a proposed patch. I'll need to ask Crawford if there is any reason he didn't check it in, and could he create a unit test.
   (10:15:44 AM) gac410: Things have been a bit slow .. .I guess summer vacations. Still waiting on gmc for installation of the weblate translation engine. And pharvey is going to help get our git repo ready for the migration.
   (10:17:10 AM) gac410: If we make some progress on those items, I'd like to announce an 8/1 (or thereabouts) freeze date for subversion repo, and we start the migration to github.
   (10:19:59 AM) gac410: How is everyone's time? Should we review the confirmed "normal" priority tasks tagged for 1.2.0 ? Or defer the normals for next meeting?
   (10:20:56 AM) JulianLevens1: I&#700;m struggling for time
   (10:21:05 AM) ModAcOst: I think it&#700;s better to defer
   (10:21:16 AM) gac410: One enhancement that I think is very low hanging is Item10824. Support the HEAD method. AndrewJones posted a small patch.
   (10:22:41 AM) gac410: Okay. Lets wrap this up. Thanks all for attending. I'll work on minutes and agend for nex meeting Monday Aug 4 Same time
   (10:23:59 AM) ModAcOst: great
   (10:24:40 AM) gac410: If we make any progress in the meantime on reorganizing the git repo, I'll get an announcement out about freezing svn.

Topic revision: r4 - 21 Jul 2014, GeorgeClark
The copyright of the content on this website is held by the contributing authors, except where stated elsewhere. See Copyright Statement. Creative Commons License    Legal Imprint    Privacy Policy