Version 48 (modified by allison, 13 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, no ticket (applied to trunk in r34503)
  • r34509-fixtestnumber.diff
  • r34826-tt121-eval-t-todo.diff, TT #121 (applied to trunk in r35149, in future don't apply unrelated patches to a branch, it just makes the branch more difficult to merge)
  • r34837-tt123-runtime.diff, TT #123 (applied to trunk in r35734, in future submit patch files not links to a branch revision number)
  • r34840-tt123-2.diff, TT #123 (applied to trunk in r35735, r35737, r35738, in future submit patch files not links to a branch revision number, and test your changes before committing)

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) include in the patch: r34479-rt56544-install_files.diff, r34932-rt56996+tt127.diff, r34944-tt128.diff
  • extensive changes for generating MANIFEST.generated file, RT #40817 (develop a single patch with all these changes, also must resolve outstanding problems mentioned in the ticket before can be applied to trunk) include in patch: r34716-rt40817-track-generated-files1.diff, r34743-rt40817-track-generated-files3.diff, r34747-rt40817-track-generated-files4-revert.diff, r34900-rt40817-track-generated-files6-MANIFEST.diff, r34901-rt40817-track-generated-files7-MANIFEST2.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)
  • r34575-tt101-add-parrot_utils-tests.diff, TT #101, (barebones test files, develop more fully)
  • r34578-add_missing_files.diff, no ticket, (dan_ops definitely shouldn't be installed, perl6 ops should be in perl6_group, pdb should be installed why deleting it?)
  • r34589-jkeenan-MANIFEST-SKIP.diff, no ticket (jim had to make these fixes because you forgot to regenerate the MANIFEST and MANIFEST.SKIP, updates to the manifest files should be included in the original patches)
  • r34724-jkeenan-gen_platform-01-t.diff, no ticket (jim made these fixes to your code, include them in the original patch)
  • r34740-rt40817-track-generated-files2.diff, no ticket (like MANIFEST and MANIFEST.SKIP, MANIFEST.generated should still be checked into subversion, for permanent record and for comparison with file generated during config and build)
  • r34758-rt40817-track-generated-files5.diff, RT #40817 (kludging in a 'TESTING' key is a hack, find another way to solve the problem)
  • r34933-tt126.diff, TT #126 (we have libraries and include files stored in different places to keep them separate, don't conflate them)
  • r34945-tt126-r34933-fixerrmsg.diff, TT #126 (makes no sense without earlier changes, which were rejected)
  • r34934-metadata.diff, no ticket (fix properties in original patches)
  • r34935-cuddled_else.diff, no ticket (fix cuddled else in original patch)
  • r34939-windows_slashes.diff, no ticket (use Perl File libraries instead of manually substituting slashes)
  • r34941-tt129.diff, TT #129 (no automatic extension adding)
  • r34485-rt56996-fhs-runtime.diff, RT #56996 (partially rejected, see alternate implementation in r35778)