Version 22 (modified by allison, 5 years ago)

--

Applied, with some modifications for architectural consistency and acceptable code quality. Ticket resolved. Comments inline and attached to relevant tickets:

  • 57548-CONDITIONED_LINE_enh.patch, RT #57548 (Unrelated changes mixed into CONDITIONED_LINE changes rejected. Willing to review them as a separate patch in a new ticket.)
  • tt94-pod2man.diff, TT #94 (applied to branch in r34478, applied to trunk in r34789)
  • rt57006-opengl-cyg.diff, RT #57006 (applied to branch in r34480, applied to trunk in r34940)
  • rt56998-cygdll_versioning.diff, RT #56998 (applied to branch in r34483, applied to trunk in r34940)
  • rt51944-README_cygwin.diff, RT #51994 (applied to branch in r34484, applied to trunk in r34718)
  • pod2man.diff, Ticket number? (applied to trunk in r34503)
  • r34509-fixtestnumber.diff

Held for extensive platform testing:

  • rt39742-installed-conflict.diff, TT #30 (applied to branch in r34477)
  • new install locations, with related searchpath changes (group these tickets/patches together into one patch for review and testing, should apply cleanly to trunk) r34479-rt56544-install_files.diff, r34485-rt56996-fhs-runtime.diff

Rejected (see reasons inline, and on relevant tickets if there are any):

  • pdd30install_stage3-unapplied.tar.bz2, TT #30 (must be split into separate patches and separate tickets)
  • r34481-rt58034-config_args.diff, RT #58034 (reject automatic run of reconfig, consult jkeenan on storing config args)
  • r34482-rt56554-make-install-lang.diff (no on target to install all languages, language developers choose their own install process), also r34510-workaround_HLLNAME.diff
  • r34502-prepare-rest.diff, no ticket (too many changes bundled together, skip the version change, put the CONDITIONED_LINE changes in a single patch, put the Zcode changes in a single patch, put any additions to RT #56554 into a patch together with that first set of changes)
  • r34505-dotnet-leftovers.diff, no ticket (put the CONDITIONED_LINE changes in the patch with the other CONDITIONED_LINE changes, put the dotnet language makefile install changes in a patch and give it to the language maintainer)
  • r34508-trailingspace.diff (fix the trailing spaces in the original patch or patches)
  • r34567-tt97-test-installables.diff, TT #97 (barebones test file, make a complete test)
  • r34570-rm-cygchkdll.diff, no ticket', (combine into one patch file together with the changes that make the configure step unnecessary)

To review

r34575-tt101-add-parrot_utils-tests.diff
r34578-add_missing_files.diff
r34589-jkeenan-MANIFEST-SKIP.diff
r34716-rt40817-track-generated-files1.diff
r34724-jkeenan-gen_platform-01-t.diff
r34740-rt40817-track-generated-files2.diff
r34743-rt40817-track-generated-files3.diff
r34747-rt40817-track-generated-files4-revert.diff
r34758-rt40817-track-generated-files5.diff
r34826-tt121-eval-t-todo.diff
r34837-tt123-runtime.diff
r34840-tt123-2.diff
r34900-rt40817-track-generated-files6-MANIFEST.diff
r34901-rt40817-track-generated-files7-MANIFEST2.diff
r34932-rt56996+tt127.diff
r34933-tt126.diff
r34934-metadata.diff
r34935-cuddled_else.diff
r34939-windows_slashes.diff
r34941-tt129.diff
r34944-tt128.diff
r34945-tt126-r34933-fixerrmsg.diff