CHANGES #2

# Change User Description Committed
#2 28901 marc_tooley ""
#1 5835 marc_tooley VCP branch to prep for incoming bugfixes for the Source/svn.pm module.
//guest/perforce_software/revml/CHANGES
#86 5087 Barrie Slaymaker - VCPDEBUG=1 can be used when running tests
#85 5084 Barrie Slaymaker - VCP::Driver chdir()s to start_dir before blowing away working dir
  in order to prevent File::Path from complaining about not being
  able to find the current working directory.  (Reported by knewt).
#84 5082 Barrie Slaymaker - VCP::Source tells VCP::Rev to uncache the source to allow
  the source instance to be DESTROYed and thus clean up its
  working files.
#83 5081 Barrie Slaymaker - VCP::Utils::cvs only tries cvs login if :pserver: and password
  field is missing.
#82 5077 Barrie Slaymaker - Updates to build system from Autrijus
#81 3640 Barrie Slaymaker - xmllint no longer require to build UI
- UI now offers multiple choices where appropriate
#80 3548 John Fetkovich Load default filters when vcp interactive ui is run.
#79 3104 John Fetkovich Added handling of VCPP4LICENSE environment variable in
       Dest::p4.pm.  If that's present and pointing to a readable
       file when a p4 daemon is started, a symlink will be created in
       the (possibly newly created) p4root directory to point to the
       p4 license file pointed to by VCPP4LICENSE.  The 'make' target
       test_all_p4_versions will cycle through each version of p4 and
       p4d contained in the 'p4versions' directory, in both
       unlicensed and (if VCPP4LICENSE present) licensed mode.
#78 3081 Barrie Slaymaker Get cvs->p4 propogation branches with multiple tags working
       to spec.
#77 3062 John Fetkovich Added capability to test all p4 versions
#76 3020 Barrie Slaymaker Note some changes, prevent hires debugging when "all" is
       provided and "hires" is not.
#75 2927 John Fetkovich edit CHANGES file for previous change.
#74 2913 Barrie Slaymaker Put a user id on each change
#73 2911 Barrie Slaymaker Document changes, add a few TODOs
#72 2802 John Fetkovich Added a source_repo_id to each revision, and repo_id to each
Source and Dest.  The repo_ids include repository type
(cvs,p4,revml,vss,...) and the repo_server fields.  Changed the
$self->...->set() and $self->...->get() lines in VCP::Dest::* to
pass in a conglomerated key value, by passing in the key as an
ARRAY ref.  Also various restructuring in VCP::DB.pm,
VCP::DB_file.pm and VCP::DB_file::sdbm.pm related to this
change.
#71 2744 John Fetkovich Change description for adding source_* fields
#70 2721 Barrie Slaymaker VCP::Dest::p4 now uses RevMapDB
#69 2710 Barrie Slaymaker Note .vop Options section availability
#68 2695 Barrie Slaymaker Prepare to make r_...
and ch_... labels optional and
       configurable by demoting them from VCP::Dest::* to tha
       single, not selected-by-default VCP::Filter::addlabels.
       This is to increase performance
#67 2683 John Fetkovich VCPPROFILETIME environment var now controls if unix 'time'
       dumps elapsed times into profile log.
#66 2682 Barrie Slaymaker more underscorification documentation.
#65 2666 John Fetkovich Analyze vcp profiling output files (created when vcp is run with VCPPROFILE
       environment var is set to filename to write to.)
#64 2663 Barrie Slaymaker Fix mtime bug in VCP::Dest::cvs in branching code
       Improve temp directory management
#63 2641 Barrie Slaymaker Add --run-p4d option to VCP::{Source,Dest}::p4.
       Implement port hunting and p4d up & ready detection for
       vcp-launched p4ds.
#62 2640 Barrie Slaymaker VCP::Source::revml now supports revision name wildcard matching.
#61 2636 Barrie Slaymaker Note test suite refactoring (no real detail, that's not
       important for a CHANGES file)
#60 2332 Barrie Slaymaker Add --init-p4d and --delete-p4d-dir options
#59 2301 Barrie Slaymaker A chain of plugins instead of source & dest
#58 2293 Barrie Slaymaker Update CHANGES, TODO, improve .vcp files, add --init-cvs
#57 2043 Barrie Slaymaker note branch support for cvs, source::p4
#56 2014 Barrie Slaymaker Give helpful error messages if the vcp command can't read/write
       RevML due to a missing required module.
#55 2009 Barrie Slaymaker lots of fixes, improve core support for branches and VCP::Source::cvs
       now supports branches.
#54 2006 Barrie Slaymaker more preparations for branching support,
       handling of cvs :foo:... CVSROOT specs,
       misc fixes, improvements
#53 1998 Barrie Slaymaker Initial, revml and core VCP support for branches
#52 1860 Barrie Slaymaker note the date of the 0.3 release
#51 1855 Barrie Slaymaker Major VSS checkin.
 Works on Win32
#50 1814 Barrie Slaymaker Don't test for all keys missing: it's not legal any more
#49 1809 Barrie Slaymaker VCP::Patch should ignore lineends
#48 1741 Barrie Slaymaker Drop extraneous entry "beatonit" from MANIFEST
#47 1740 Barrie Slaymaker Note 0.28 release, finish adding dist/vcp.pl
#46 1728 Barrie Slaymaker CVS on win32, minor bugfixes
#45 1506 Barrie Slaymaker tweaks
#44 1367 Barrie Slaymaker lots of docco updates
#43 1366 Barrie Slaymaker Require Regexp::Shellish 0.93 for '...' support
#42 1359 Barrie Slaymaker Revamp the help system, clean up and add POD
#41 1358 Barrie Slaymaker Win32 changes
#40 1330 Barrie Slaymaker Ignore cvs lock mgmt warnings in VCP::Source::cvs.
#39 1175 Barrie Slaymaker Implement VCP::Patch, roll 0.26 release.
#38 1174 Barrie Slaymaker Add and use VCP::DiffFormat
#37 1171 Barrie Slaymaker Switch to using Text::Diff
#36 1069 Barrie Slaymaker bump to 0.25, hide spurious test output
#35 1055 Barrie Slaymaker add sorting, revamp test suite, misc cleanup.
 Dest/revml is
not portable off my system yet (need to release ...::Diff)
#34 1022 Barrie Slaymaker Perl "$foo\_..." => "${foo}_..." cleanup by
Peter Prymmer <PPrymmer@factset.com>.
#33 825 Barrie Slaymaker test, handle case where no revs are transferred and
VCP::Dest::*::handle_footer() blew up.
#32 817 Barrie Slaymaker more updates
#31 816 Barrie Slaymaker update CHANGES
#30 723 Barrie Slaymaker VCP::Dest::cvs tuning and cvs and p4 bugfixes
#29 721 Barrie Slaymaker Add File::Temp to PREREQ_PM, older perls don't come with it.
#28 720 Barrie Slaymaker Fix handling of $r->comment in VCP::Dest::cvs
#27 719 Barrie Slaymaker vcp 0.221
#26 705 Barrie Slaymaker Release 0.22.
#25 703 Barrie Slaymaker VCP::Source::p4 now uses VCP::Utils::p4::parse_p4_repo_spec()
#24 701 Barrie Slaymaker Fixed VCP::Dest::p4 re-rooting problem, further t/* cleanup
#23 699 Barrie Slaymaker test suite cleanup
#22 695 Barrie Slaymaker Cleaned up support for binary files in VCP::Dest::revml and
altered test suite to deal with it better.  Added some thoughts
to the TODO file.
#21 694 Barrie Slaymaker cleanup up warning about undef var in VCP::Utils::p4.
#20 692 Barrie Slaymaker Add VCP::Utils::p4 and use it to get VCP::Dest::p4 to create it's
own client view as needed.
#19 689 Barrie Slaymaker reinstate -f behavior as the default for VCP::Source::cvs, clean
up -D --> -d doco.
#18 688 Barrie Slaymaker Fixed docos for --debug.
#17 687 Barrie Slaymaker remove -f, tweak deduce_rev_root
#16 674 Barrie Slaymaker Fix paths passed to p4 labelsync in VCP::Des::P4, reported by
david d zuhn <zoo@bravara.com>
#15 669 Barrie Slaymaker 0.1 Wed Jul  4 00:27:35 EDT 2001
Fix VCP::Dest::p4 to take the filespec from the p4:<dest> spec and
use it as the rev_root.  No --rev-root option at this time, not sure
if it's needed.  Reported by david d zuhn <zoo@bravara.com>.
#14 630 Barrie Slaymaker Fix bug in CVS log file parsing that made it think it was always
seeing the same file, different revisions over and over again.
Reported by Matthew Attaway.
#13 627 Barrie Slaymaker Beef up CVS log file parsing.
#12 620 Barrie Slaymaker Underscorify CVS tags, only warn about undeleted files if
debugging.
#11 619 Barrie Slaymaker Avoid using p4 print -s, it puts linebreaks in every 4098
characters or so.
#10 615 Barrie Slaymaker Detect p4d <= 99.2 and skip tests.
 Fix a use strict problem.
Both reported by Nick Ing-Simmons.
#9 608 Barrie Slaymaker Lots of changes to get vcp to install better, now up to 0.066.
Many thanks to Matthew Attaway for testing & suggestions.
#8 483 Barrie Slaymaker Makefile.PL now requires at least XML::AutoWriter v0.36
#7 482 Barrie Slaymaker Modified CHANGES to reflect last submit
#6 480 Barrie Slaymaker 0.06 Wed Dec 20 23:19:15 EST 2000
   - bin/vcp: Added --versions, which loads all modules and checks them
     for a $VERSION and print the results out.  This should help with
     diagnosing out-of-sync modules.
   - Added $VERSION vars to a few modules :-).  Forgot to increment any
     $VERSION strings.
   - VCP::Dest::cvs: The directory "deeply" was not being `cvs add`ed on
     paths like "a/deeply/nested/file", assuming "deeply" had no files
     in it.
   - VCP::Dest::revml: fixed a bug that was causing files with a lot of
     linefeeds to be emitted in base64 instead of deltaed.  This means
     most text files.
   - Various minor cleanups of diagnostics and error messages, including
     exposing "Can't locate Foo.pm" when a VCP::Source or VCP::Dest
     module depends on a module that's not installed, as reported by
     Jeff Anton.
#5 478 Barrie Slaymaker 0.05 Mon Dec 18 07:27:53 EST 2000
   - Use `p4 labels //...@label` command as per Rober Cowham's suggestion, with
     the '-s' flag recommended by Christopher Siewald and
     Amaury.FORGEOTDARC@atsm.fr.  Though it's actually something like

       vcp: running /usr/bin/p4 -u safari -c safari -p localhost:5666 -s files
       //.../NtLkly //...@compiler_a3 //.../NtLkly //...@compiler_may3

     and so //on //for 50 parameters to get the speed up.  I use the
     //.../NtLkly "file" as //a separator between the lists of files in various
     //revisions.  Hope nobody has any files named that :-).  What I should do
     is choose a random label that doesn't occur in the labels list, I guess.
   - VCP::Source::revml and VCP::Dest::revml are now binary, control code, and
     "hibit ASCII" (I know, that's an oxymoron) clean.  The <comment>, <delta>,
     and <content> elements now escape anything other than tab, line feed,
     space, or printable chars (32 <= c <= ASCII 126) using a tag like '<char
     code="0x09">'.  The test suite tests all this.  Filenames should also
     be escaped this way, but I didn't get to that.
   - The decision whether to do deltas or encode the content in base64 is now
     based on how many characters would need to be escaped.
   - We now depend on the users' diff program to have a "-a" option to force it
     to diff even if the files look binary to it.  I need to use Diff.pm and
     adapt it for use on binary data.
   - VCP::Dest::cvs now makes sure that no two consecutive revisions of the
     same file have the same mod_time.  VCP::Source::p4 got so fast at pulling
     revisions from the repositories the test suite sets up that CVS was not
     noticing that files had changed.
   - VCP::Plugin now allows you to set a list of acceptable result codes, since
     we now use p4 in ways that make it return non-zero result codes.
   - VCP::Revs now croaks if you try to add two entries of the same VCP::Rev
     (ie matching filename and rev_id).
   - The <type> tag is now limited to "text" or "binary", and is meant to
     pass that level of info between foreign repositories.
   - The <p4_info> on each file now carries the one line p4 description of
     the file so that p4->p4 transferes can pick out the more detailed
     info.  VCP::Source::p4, VCP::Dest::p4 do this.
   - VCP::{Source,Dest}::{p4,cvs} now set binaryness on added files properly,
     I think.  For p4->p4, the native p4 type is preserved.  For CVS sources,
     seeing the keyword substitution flag 'o' or 'b' implies binaryness, for
     p4, seeing a filetype like qr/u?x?binary/ or qr/x?tempobj/ or "resource"
     implies binaryness (to non-p4 destinations).  NOTE: Seeing a 'o' or 'b'
     in a CVS source only ends up setting the 'b' option on the destination.
     That should be ok for most uses, but we can make it smarter for cvs->cvs
     transfers if need be.
#4 473 Barrie Slaymaker 0.04 Tue Dec 12 00:15:57 EST 2000
   - Reorg of VCP::Source::p4
     - One large filelog command is run instead of many small ones.
       This takes advantage of the -m option to make sure enough changes
       are listed.  Many extra revisions of most files are probably
       listed, but listing and ignoring them is quicker than spawning p4
       over and over.  Wish p4 filelog had a revision range...
     - it now doesn't suck the entire filelog output in to memory, it
       parses it line by line as it's emitted from the `p4 filelog`
     - `p4 print` is now used to print a bunch of files at once, using
       the header line to separate one file from the next, kind of like
       splitting a mime-encoded message.  There's a very slight chance
       that it will misjudge the boundary between two files if a file
       happens to have a line that looks very much like the header line
       for the next file.  This is pretty unlikely and I'll fix it if it
       crops up.  I could batch them more, right now it never puts two
       revisions of the same filename in the same batch, for no really
       good reason.  Another method might be to batch 25 or 50 revs each
       time.
     - it turns out there's a problem spawning multiple p4 commands at
       the same time against the same p4d (p4d is 99.2, FWIW).  Or at
       least running large `p4 files ...` while there's a large `p4
       filelog` still also running.
     - filelog lines beginning with "... ..." are now ignored.  These
       are notifications of copy, branch, and integrate events that we
       don't yet do anything with.
     - deleted cur() and P4_CUR
     - deleted P4_IS_INCREMENTAL
   - Made an assertion in VCP::Dest::revml::handle_rev() a little
     clearer
   - Added some ok(1) calls to 90p4.t to make it easier to figure out
     which child process is whining or aborting
   - Made the message that's printed when a subcommand emits unexpected
     output say "stderr" instead of "stdout".
   - Cleaned up documentation for VC::Plugin::work_path().
#3 470 Barrie Slaymaker - Cleaned up VCP::Source::p4 a bit.
 It doesn't whine as much now
    when it sees what it considers to be old news in the log file.
  - Added an easy way to monitor the commands being issued to a
    repository: simply add "::cmd" to the debug spec for that source:

       vcp -d Source::cvs::cvs
       vcp -d Dest::p4::p4

  - The next step is to use the -m option to p4 filelog to speed
    things up.
#2 468 Barrie Slaymaker - VCP::Dest::p4 now does change number aggregation based on the
  comment field changing or whenever a new revision of a file with
  unsubmitted changes shows up on the input stream.  Since revisions of
  files are normally sorted in time order, this should work in a number
  of cases.  I'm sure we'll need to generalize it, perhaps with a time
  thresholding function.
- t/90cvs.t now tests cvs->p4 replication.
- VCP::Dest::p4 now doesn't try to `p4 submit` when no changes are
  pending.
- VCP::Rev now prevents the same label from being applied twice to
  a revision.  This was occuring because the "r_1"-style label that
  gets added to a target revision by VCP::Dest::p4 could duplicate
  a label "r_1" that happened to already be on a revision.
- Added t/00rev.t, the beginnings of a test suite for VCP::Rev.
- Tweaked bin/gentrevml to comment revisions with their change number
  instead of using a unique comment for every revision for non-p4
  t/test-*-in-0.revml files.  This was necessary to test cvs->p4
  functionality.
#1 467 Barrie Slaymaker Version 0.01, initial checkin in perforce public depot.