package VCP::Dest ; =head1 NAME VCP::Dest - A base class for VCP destinations =head1 SYNOPSIS =head1 DESCRIPTION =head1 EXTERNAL METHODS =over =for test_scripts t/01sort.t =cut use strict ; use Carp ; use UNIVERSAL qw( isa ) ; use VCP::Revs ; use VCP::Debug qw(:debug) ; use vars qw( $VERSION $debug ) ; $VERSION = 0.1 ; $debug = 0 ; use base 'VCP::Plugin' ; use fields ( 'DEST_HEADER', ## Holds header info until first rev is seen. 'DEST_SORT_SPEC', ## ARRAY of field names to sort by 'DEST_SORT_KEYS', ## HASH of sort keys, indexed by name and rev. 'DEST_COMMENT_TIMES', ## The average time of all instances of a comment 'DEST_DEFAULT_COMMENT', ## The comment to use when a comment is undefined ## This is used when presorting/merging so ## that comment will still be used to ## compare when selecting the next rev to ## merge, otherwise it would be removed as ## a sporadic field. ) ; use VCP::Revs ; =item new Creates an instance, see subclasses for options. The options passed are usually native command-line options for the underlying repository's client. These are usually parsed and, perhaps, checked for validity by calling the underlying command line. =cut sub new { my $class = shift ; $class = ref $class || $class ; my VCP::Dest $self = $class->SUPER::new( @_ ) ; ## rev_id is here in case the change id isn't, ## name is here for VSS deletes, which have no other data. $self->set_sort_spec( "presort,change_id,time,avg_comment_time,comment,name,rev_id" ) ; return $self ; } =back ############################################################################### =head1 SUBCLASSING This class uses the fields pragma, so you'll need to use base and possibly fields in any subclasses. =head2 SUBCLASS API These methods are intended to support subclasses. =over =item digest $self->digest( "/tmp/readers" ) ; Returns the Base64 MD5 digest of the named file. Used to compare a base rev (which is the revision *before* the first one we want to transfer) of a file from the source repo to the existing head rev of a dest repo. The Base64 version is returned because that's what RevML uses and we might want to cross-check with a .revml file when debugging. =cut sub digest { shift ; ## selfless little bugger, isn't it? my ( $path ) = @_ ; require Digest::MD5 ; my $d= Digest::MD5->new ; open DEST_P4_F, "<$path" or die "$!: $path" ; $d->addfile( \*DEST_P4_F ) ; my $digest = $d->b64digest ; close DEST_P4_F ; return $digest ; } =item compare_base_revs $self->compare_base_revs( $rev ) ; Checks out the indicated revision fromt the destination repository and compares it (using digest()) to the file from the source repository (as indicated by $rev->work_path). Dies with an error message if the base revisions do not match. Calls $self->checkout_file( $rev ), which the subclass must implement. =cut sub compare_base_revs { my VCP::Dest $self = shift ; my ( $rev ) = @_ ; ## This block should only be run when transferring an incremental rev. ## from a "real" repo. If it's from a .revml file, the backfill will ## already be done for us. ## Grab it and see if it's the same... my $source_digest = $self->digest( $rev->work_path ) ; my $backfilled_path = $self->checkout_file( $rev ); my $dest_digest = $self->digest( $backfilled_path ); die( "vcp: base revision\n", $rev->as_string, "\n", "differs from the last version in the destination p4 repository.\n", " source digest: $source_digest (in ", $rev->work_path, ")\n", " dest. digest: $dest_digest (in ", $backfilled_path, ")\n" ) unless $source_digest eq $dest_digest ; } =item header Gets/sets the $header passed to handle_header(). Generally not overridden: all error checking is done in new(), and no output should be generated until output() is called. =cut sub header { my VCP::Dest $self = shift ; $self->{DEST_HEADER} = shift if @_ ; return $self->{DEST_HEADER} ; } =back =head2 SUBCLASS OVERLOADS These methods are overloaded by subclasses. =over =item backfill $dest->backfill( $rev ) ; Checks the file indicated by VCP::Rev $rev out of the target repository if this destination supports backfilling. Currently, only the revml destination does not support backfilling. The $rev->{workpath} must be set to the filename the backfill was put in. This is used when doing an incremental update, where the first revision of a file in the update is encoded as a delta from the prior version. A digest of the prior version is sent along before the first version delta to verify it's presence in the database. So, the source calls backfill(), which returns TRUE on success, FALSE if the destination doesn't support backfilling, and dies if there's an error in procuring the right revision. If FALSE is returned, then the revisions will be sent through with no working path, but will have a delta record. MUST BE OVERRIDDEN. =cut sub backfill { my VCP::Dest $self = shift ; my ( $r ) = @_; die ref( $self ) . "::checkout_file() not found for ", $r->as_string, "\n" unless $self->can( "checkout_file" ); my $work_path = $self->checkout_file( $r ); link $work_path, $r->work_path or die "$! linking $work_path to ", $r->work_path; unlink $work_path or die "$! unlinking $work_path"; } =item handle_footer $dest->handle_footer( $footer ) ; Does any cleanup necessary. Not required. Don't call this from the override. =cut sub handle_footer { my VCP::Dest $self = shift ; return ; } =item handle_header $dest->handle_header( $header ) ; Stows $header in $self->header. This should only rarely be overridden, since the first call to handle_rev() should output any header info. =cut sub handle_header { my VCP::Dest $self = shift ; my ( $header ) = @_ ; $self->header( $header ) ; return ; } =item handle_rev $dest->handle_rev( $rev ) ; Outputs the item referred to by VCP::Rev $rev. If this is the first call, then $self->none_seen will be TRUE and any preamble should be emitted. MUST BE OVERRIDDEN. Don't call this from the override. =cut sub handle_rev { my VCP::Dest $self = shift ; die ref( $self ) . "::handle_rev() not found, Oops.\n" ; } =back =head3 Sorting =over =item set_sort_spec $dest->set_sort_spec( @key_names ) ; @key_names specifies the list of fields to sort by. Each element in the array may be a comma separated list. Such elements are treated as though each name was passed in it's own element; so C<( "a", "b,c" )> is equivalent to C<("a", "b", "c")>. This eases command line parameter parsing. Sets the sort specification, checking to make sure that the field_names have corresponding parse_sort_field_... handlers in this object. Legal field names include: name, change, change_id, rev, rev_id, comment, time. If a field is missing from all revs, it is ignored, however at least one of rev_id, change, or time *must* be used. Default ordering is by - change_id (compared numerically using <=>, for now) - time (commit time: simple numeric, since this is a simple number) - comment (alphabetically, case sensitive) This ordering benefits change number oriented systems while preserving commit order for non-change number oriented systems. If change_id is undefined in either rev, it is not used. If time is undefined in a rev, the value "-1" is used. This causes base revisions (ie digest-only) to precede real revisions. That's not always good, though: one of commit time or change number should be defined! Change ids are compared numerically, times by date order (ie numerically, since time-since-the-epoch is used internally). Comments are compared alphabetically. Each sort field is split in to one or more segments, see the appropriate parse_sort_field_... documentation. Here's the sorting rules: - Revisions are compared field by field. - The first non-equal field determines sort order. - Fields are compared segment by segment. - The first non-equal segment determines sort order. - A not-present segment compares as less-than any other segment, so fields that are leading substrings of longer fields come first, and not-present fields come before all present fields, including empty fields. =cut sub set_sort_spec { my VCP::Dest $self = shift ; my @spec = split ',', join ',', @_ ; # for ( @spec ) { # next if /^presort\b/; # next if $self->can( "parse_sort_field_$_" ) ; # croak "Sort specification $_ is not available in ", # ref( $self ) =~ /.*:(.*)/ ; # } debug "vcp: sort spec: ", join ",", @spec if explicitly_debugging "sort" || debugging $self ; $self->{DEST_SORT_SPEC} = \@spec ; return undef ; } =item How rev_id and change_id are sorted C<change_id> or C<rev_id> are split in to segments suitable for sorting. The splits occur at the following points: 1. Before and after each substring of consecutive digits 2. Before and after each substring of consecutive letters 3. Before and after each non-alpha-numeric character The substrings are greedy: each is as long as possible and non-alphanumeric characters are discarded. So "11..22aa33" is split in to 5 segments: ( 11, "", 22, "aa", 33 ). If a segment is numeric, it is left padded with 10 NUL characters. This algorithm makes 1.52 be treated like revision 1, minor revision 52, not like a floating point C<1.52>. So the following sort order is maintained: 1.0 1.0b1 1.0b2 1.0b10 1.0c 1.1 1.2 1.10 1.11 1.12 The substring "pre" might be treated specially at some point. (At least) the following cases are not handled by this algorithm: 1. floating point rev_ids: 1.0, 1.1, 1.11, 1.12, 1.2 2. letters as "prereleases": 1.0a, 1.0b, 1.0, 1.1a, 1.1 Never returns (), since C<rev_id> is a required field. =cut sub _compile_sort_rec_indexer { my ( $rev, $spec ) = @_ ; my $code = join "", q[sub { local $_ = shift; pack '], map( $rev->pack_format( $_ ), @$spec ), q[', ], join( ", ", map $rev->index_value_expression( $_ ), @$spec ), q[}]; warn caller, $code; return eval $code or die $@; } =item sort_revs $source->dest->sort_revs( $source->revs ) ; This sorts the revisions that the source has identified in to whatever order is needed by the destination. Sorting is normally done in two passes. Each file's revisions are sorted by change, rev_id, or time, then the resulting lists of revisions are sorted in to one long list by pulling the "least" revision off the head of each list based on change, time, comment, and name. This two-phased approach ensures that revisions of a file are always in proper order no matter what order they are provided, and furthermore in an order that enables change number aggregation (which is useful even if the destination does not provide change numbers if only to do batch submits or get commit timestamps in a sensible order). This is all because, when doing change aggregation, it's necessary to put revision number last in the sort key fields. If revisions of a file have bogus timestamps, say, this can cause revisions of a file to be sorted out of order. So we sort on rev_id first to guarantee that each file's versions are in a correct order, then we do change aggregation. The sort specification "presort" is used to engage the presort-and-merge algorithm, and is in the default sort spec. There is currently no way to affect the sort order in the presort phase. =cut sub _calc_sort_recs { my VCP::Dest $self = shift ; my ( $sort_recs, $spec ) = @_; debug "vcp sort key: ", join ", ", @$spec if debugging "sort" ; if ( grep /avg_comment_time/, @$spec ) { $self->{DEST_COMMENT_TIMES} = {}; for ( @$sort_recs ) { my $r = $_->[0]; my $comment = defined $r->comment ? $r->comment : $r->is_base_rev ? "" : undef; my $time = defined $r->time ? $r->time : $r->is_base_rev ? 0 : undef; next unless defined $comment && defined $time; push @{$self->{DEST_COMMENT_TIMES}->{$comment}}, $time; } for ( values %{$self->{DEST_COMMENT_TIMES}} ) { next unless @$_; my $sum; $sum += $_ for @$_; $_ = $sum / @$_; } } my $indexer = _compile_sort_rec_indexer( $sort_recs->[0]->[0], $spec ); for ( @$sort_recs) { ## Modify each sort rec in place, don't replace the list. ## This routine is called a second time on the same ## set of sortrecs when presorting & merging, which ## happens a lot. $_->[1] = $indexer->( $_->[0] ); } } sub _remove_sporadics { my VCP::Dest $self = shift; my ( $sort_recs, $spec ) = @_; return; ## TODO!!! my @not_seen = my @seen = ( 0 ) x @$spec; my @sporadics; my @keepers = ( 0 ); ## Always keep the rev ($sort_rec[0]) my $its_ok; my $max_i = 0; for ( @$sort_recs) { $max_i = $#$_ if $#$_ > $max_i; for my $i ( 1..$#$_ ) { if ( @{$_->[$i]} && defined $_->[$i]->[0] ) { ++$seen[$i]; } else { ++$not_seen[$i]; } } } for my $i ( 1..$max_i ) { if ( $seen[$i] && $not_seen[$i] ) { push @sporadics, $i; next; } push @keepers, $i; if ( $seen[$i] && $spec->[$i-1] =~ /^(change|rev|time)/ ) { ## One of the numeric ordering fields is present. $its_ok = 1; } } if ( @sporadics ) { my @sp_desc = map "$spec->[$_] (seen $seen[$_] times, missing $not_seen[$_])\n", @sporadics; unless ( $its_ok ) { die "missing sort key", @sp_desc == 1 ? () : "s", " while sorting revisions:", @sp_desc == 1 ? ( " ", @sp_desc ) : ( "\n", map " $_", @sp_desc ), "sort keys are ", join( ", ", @$spec ), "\n"; } debug "removing sporadic sort key", @sp_desc == 1 ? ( ": ", @sp_desc ) : ( "s:\n", map " $_", @sp_desc ) if debugging; confess "not keeping revs" unless $keepers[0] == 0; @$_ = @{$_}[@keepers] for @$sort_recs; } } sub _presort_revs { my VCP::Dest $self = shift ; my ( $revs, $sort_spec ) = @_; debug "presorting revisions by ", join ", ", @$sort_spec if debugging ; debug "...categorizing and generating index" if debugging ; my %p; my $indexer; for my $r ( $revs->get ) { $indexer ||= _compile_sort_rec_indexer( $r, $sort_spec ); push @{$p{$r->name}}, [ $r, $indexer->( $r ) ]; } debug "...presorting revisions" if debugging ; my @p = map [ sort { $a->[1] cmp $b->[1] } @$_ ], values %p; debug "...done presort" if debugging ; return \@p; } sub _merge_presorted_revs { my VCP::Dest $self = shift ; my ( $presorted, $spec ) = @_; debug "merging presorted revisions by ", join ", ", @$spec if debugging ; my @p = splice @$presorted; my @sort_recs = map @$_, @p; $self->{DEST_DEFAULT_COMMENT} = ""; debug "...generating index" if debugging ; $self->_calc_sort_recs( \@sort_recs, $spec ); $self->{DEST_DEFAULT_COMMENT} = undef; ## Fill in missing time values as best as we can for things like ## VSS. if ( grep $_ eq "time", @$spec ) { debug "...filling in missing time values" if debugging ; my $found = 0; for ( @p ) { my $prev_t = "9" x 10; for ( reverse @$_ ) { my $t = $_->[0]->time; if ( defined $t ) { $prev_t = $t; } else { $_->[0]->estimated_time( $prev_t ); } } } debug "...estimated $found time values" if debugging ; } $self->_remove_sporadics( \@sort_recs, $spec ); debug "...merging presorted lists" if debugging; my @result; my %rank; debug "...ranking" if debugging; $rank{ int $_ } = keys %rank for sort { $a->[1] cmp $b->[1] } map @$_, @p; debug "......sorting first files by rank" if debugging; @p = sort { $rank{ int $a->[0] } <=> $rank{ int $b->[0] } } @p; #for my $i ( 1..$#p ) { # die "sort already broken" if $rank{ int $p[$i-1]->[0]} > $rank{ int $p[$i]->[0] }; #} debug "......merging" if debugging; while ( @p ) { push @result, (shift @{$p[0]})->[0]; if ( ! @{$p[0]} ) { ## No more revs for that file. shift @p; next if @p >= 2; } if ( @p == 1 ) { ## ...and it was the only or next to last file. push @result, map $_->[0], @{shift @p} if @p == 1; last; } ## $p[0] may need to be moved. my $r = $rank{int $p[0]->[0]}; ## Do an insertion sort using a binary search. ## After seeing it should next if $r < $rank{ int $p[1]->[0] }; # Nope. it stays where it is. my $p = shift @p; if ( $r > $rank{ int $p[-1]->[0] } ) { # hmmm, it goes on the end. Even if this is not a common # case, the binary search below does not deal well if we # don't do this. push @p, $p; next; } my ( $l, $u ) = ( 0, $#p ); # Invariants # $l is at or below the insertion point. # $u + 1 is at or above the insertion point. my $m; while ( $l < $u ) { $m = ( $l + $u ) >> 1; ## TODO: interpolate $m? if ( $r > $rank{ int $p[$m]->[0] } ) { # must insert after $m. $l = $m + 1; } else { # must insert before $m+1 $u = $m; } } #print STDERR "=$l\n"; splice @p, $l, 0, $p; #for my $i ( 1..$#p ) { # die "sort broken" if $rank{ int $p[$i]->[0]} < $rank{ int $p[$i-1]->[0] }; #} } debug "done merging" if debugging $self; @result; } sub sort_revs { my VCP::Dest $self = shift ; my ( $revs ) = @_ ; VCP::Rev::preindex(); my @spec = @{$self->{DEST_SORT_SPEC}}; if ( substr( $spec[0], 0, 7 ) eq "presort" ) { my @prespec = ( shift @spec ) =~ m/presort\((?:\s*(\w+)[,)])*\)/; @prespec = qw( change_id rev_id time ) unless @prespec; my $presorted = $self->_presort_revs( $revs, \@prespec ); $revs->set( $self->_merge_presorted_revs( $presorted, \@spec ) ); debug "done merge" if debugging ; return; } debug "generating index" if debugging ; my $sort_recs = [ map [ $_ ], $revs->get ]; $self->_calc_sort_recs( $sort_recs, \@spec ); $self->_remove_sporadics( $sort_recs, \@spec ); debug "sorting revisions" if debugging ; $revs->set( map $_->[0], sort { $a->[1] cmp $b->[1] } @$sort_recs ) ; } =item metadata_only This returns false by default, but the experimental branch_diagram destination requires only metadata. A source should look at this before going to the effort of checking out each file. =cut sub metadata_only { 0 } =back =head1 NOTES Several fields are jury rigged for "base revisions": these are fake revisions used to start off incremental, non-bootstrap transfers with the MD5 digest of the version that must be the last version in the target repository. Since these are "faked", they don't contain comments or timestamps, so the comment and timestamp fields are treated as "" and 0 by the sort routines. There is a special sortkey C<avg_comment_time> that allows revisions within the same time period (second, minute, day) to be sorted according to the average time of the comment for the revision (across all revisions with that comment). This causes changes that span more than one time period to still be grouped properly. =cut =head1 COPYRIGHT Copyright 2000, Perforce Software, Inc. All Rights Reserved. This module and the VCP package are licensed according to the terms given in the file LICENSE accompanying this distribution, a copy of which is included in L<vcp>. =head1 AUTHOR Barrie Slaymaker <barries@slaysys.com> =cut 1
# | Change | User | Description | Committed | |
---|---|---|---|---|---|
#57 | 4497 | Barrie Slaymaker |
- --rev-root documented - All destinations handle rev_root defaulting now |
||
#56 | 4487 | Barrie Slaymaker | - dead code removal (thanks to clkao's coverage report) | ||
#55 | 4483 | Barrie Slaymaker | - calls to skip_rev() are summarized to STDOUT | ||
#54 | 4021 | Barrie Slaymaker |
- Remove all phashes and all base & fields pragmas - Work around SWASHGET error |
||
#53 | 3855 | Barrie Slaymaker |
- vcp scan, filter, transfer basically functional - Need more work in re: storage format, etc, but functional |
||
#52 | 3850 | Barrie Slaymaker | - No longer stores all revs in memory | ||
#51 | 3809 | Barrie Slaymaker | - compare_base_revs() now always called with 2 parameters | ||
#50 | 3805 | Barrie Slaymaker | - VCP::Revs::fetch_files() removed | ||
#49 | 3802 | Barrie Slaymaker | - tweak whitespace | ||
#48 | 3800 | Barrie Slaymaker | - <branches> removed from all code | ||
#47 | 3769 | Barrie Slaymaker | - avg_comment_time sort key removed | ||
#46 | 3706 | Barrie Slaymaker | - VCP gives some indication of output progress (need more) | ||
#45 | 3460 | Barrie Slaymaker |
- Revamp Plugin/Source/Dest hierarchy to allow for reguritating options in to .vcp files |
||
#44 | 3429 | Barrie Slaymaker |
- Refactor db_location() into VCP::Plugin so VCP::Source::vss will be able to use it. |
||
#43 | 3409 | Barrie Slaymaker | - Minor doc and code readability improvements | ||
#42 | 3155 | Barrie Slaymaker |
Convert to logging using VCP::Logger to reduce stdout/err spew. Simplify & speed up debugging quite a bit. Provide more verbose information in logs. Print to STDERR progress reports to keep users from wondering what's going on. Breaks test; halfway through upgrading run3() to an inline function for speed and for VCP specific features. |
||
#41 | 3133 | Barrie Slaymaker |
Make destinations call back to sources to check out files to simplify the architecture (is_metadata_only() no longer needed) and make it more optimizable (checkouts can be batched). |
||
#40 | 3129 | Barrie Slaymaker |
Stop calling the slow Cwd::cwd so much, use start_dir instead. |
||
#39 | 3120 | Barrie Slaymaker | Move changeset aggregation in to its own filter. | ||
#38 | 3115 | Barrie Slaymaker |
Move sorting function to the new VCP::Filter::sort; it's for testing and reporting only and the code was bloating VCP::Dest and limiting VCP::Rev and VCP::Dest optimizations. Breaks test suite in minor way. |
||
#37 | 3096 | Barrie Slaymaker | Tuning | ||
#36 | 3087 | Barrie Slaymaker | Improve diagnostics | ||
#35 | 3084 | Barrie Slaymaker | Minor improvement to reporting. | ||
#34 | 3077 | Barrie Slaymaker | remove debugging output | ||
#33 | 3076 | Barrie Slaymaker | Improve change aggregation | ||
#32 | 3059 | Barrie Slaymaker | Minor cleanup of warning about undefined variable usage | ||
#31 | 3046 | Barrie Slaymaker | Fix revision sorting | ||
#30 | 3008 | John Fetkovich |
make state database files go under vcp_state in the program start directory (start_dir) instead of start_dir itself. Also escape periods (.) from the database directory as well as the characters already escaped. |
||
#29 | 2959 | John Fetkovich |
added dump method to lib/VCP/DB_File/sdbm.pm to dump keys => values from a sdbm file. removed similar code from bin/dump_head_revs, bin/dump_rev_map and bin/dump_main_branch_id and called this method instead. also made parse_files_and_revids_from_head_revs_db sub in TestUtils to use in test suites instead of parse_files_and_revids_from_p4_files et. al. |
||
#28 | 2928 | John Fetkovich |
Added empty sub to VCP::Utils.pm to check for empty or undefined strings. Added a couple of calls to it in Dest.pm. |
||
#27 | 2926 | John Fetkovich |
remove --state-location switch add --db-dir and --repo-id switches build state location from concatenation of those two. |
||
#26 | 2899 | Barrie Slaymaker |
Implement a natural sort that organizes the revs in to trees and then builts the submittal order by poping the first root off the trees and then sorting any child revs in to the roots list. |
||
#25 | 2873 | Barrie Slaymaker | Add MainBranchIdDB and a dump util. | ||
#24 | 2808 | Barrie Slaymaker | Pass source_repo_id in to last_rev_in_filebranch | ||
#23 | 2800 | Barrie Slaymaker | Get --continue working in cvs->foo transfers. | ||
#22 | 2725 | Barrie Slaymaker | Start using HeadRevs.pm. | ||
#21 | 2720 | Barrie Slaymaker | Factor RevMapDB code up in to VCP::Dest. | ||
#20 | 2713 | Barrie Slaymaker | Factor RevMapDB management up in to VCP::Dest | ||
#19 | 2330 | Barrie Slaymaker | Silence warnings in corner condition of transferring one file. | ||
#18 | 2324 | Barrie Slaymaker |
Take branch_id in to account in presort stage so that branched files with the same name get treated as independant files. |
||
#17 | 2241 | Barrie Slaymaker | RCS file scanning improvements, implement some of -r | ||
#16 | 2235 | Barrie Slaymaker | Debugging cvs speed reader. | ||
#15 | 2233 | Barrie Slaymaker | debug | ||
#14 | 2232 | Barrie Slaymaker | Major memory and sort speed enhancements. | ||
#13 | 2228 | Barrie Slaymaker | working checkin | ||
#12 | 2198 | Barrie Slaymaker | Minor bugfix for single file mode. | ||
#11 | 2154 | Barrie Slaymaker | Speed up sorting | ||
#10 | 2042 | Barrie Slaymaker | Basic source::p4 branching support | ||
#9 | 2009 | Barrie Slaymaker |
lots of fixes, improve core support for branches and VCP::Source::cvs now supports branches. |
||
#8 | 1855 | Barrie Slaymaker |
Major VSS checkin. Works on Win32 |
||
#7 | 1822 | Barrie Slaymaker |
Get all other tests passing but VSS. Add agvcommenttime sort field. |
||
#6 | 1809 | Barrie Slaymaker | VCP::Patch should ignore lineends | ||
#5 | 1055 | Barrie Slaymaker |
add sorting, revamp test suite, misc cleanup. Dest/revml is not portable off my system yet (need to release ...::Diff) |
||
#4 | 827 | Barrie Slaymaker | Add a test for and debug p4->cvs incremental exports. | ||
#3 | 628 | Barrie Slaymaker | Cleaned up POD in bin/vcp, added BSD-style license. | ||
#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. |