svn commit: r1922161 - in /subversion/branches/apply-processor: ./ .asf.yaml subversion/libsvn_client/merge_processor.c

[email protected]
Newsgroups gmane.comp.version-control.subversion.svn
Message-ID <[email protected]>
Author: rinrab
Date: Wed Nov 27 13:05:03 2024
New Revision: 1922161

URL: http://svn.apache.org/viewvc?rev=1922161&view=rev
Log:
On the 'apply-processor' branch: Merge changes from trunk@1922159, resolving
conflicts in merge.c manually, due to specific changes in the scope of this
branch.

Modified:
    subversion/branches/apply-processor/   (props changed)
    subversion/branches/apply-processor/.asf.yaml
    subversion/branches/apply-processor/subversion/libsvn_client/merge_processor.c

Propchange: subversion/branches/apply-processor/
------------------------------------------------------------------------------
  Merged /subversion/trunk:r1922128-1922136,1922139-1922159

Modified: subversion/branches/apply-processor/.asf.yaml
URL: http://svn.apache.org/viewvc/subversion/branches/apply-processor/.asf.yaml?rev=1922161&r1=1922160&r2=1922161&view=diff
==============================================================================
--- subversion/branches/apply-processor/.asf.yaml (original)
+++ subversion/branches/apply-processor/.asf.yaml Wed Nov 27 13:05:03 2024
@@ -19,5 +19,17 @@
 # .asf.yml -- ASF infra services configuration
 #
 
-notifications:
-  jobs: [email protected]
+# GitHub Action notifications, see 
+# https://github.com/apache/infrastructure-asfyaml?tab=readme-ov-file
+#
+# However, according to discussion with Humbedooh on Slack the-asf#Infra,
+# it require a r/w Git repository, so the below doesn't work.
+#
+# Humbedooh has kindly helped us hard-code a one-off notification which
+# doesn't rely on .asf.yaml.
+#
+# Commenting out, but keeping in place as a sticky-note for future changes.
+#
+# For reference, this is the hard-coded configuration:
+#notifications:
+#  jobs: [email protected]

Modified: subversion/branches/apply-processor/subversion/libsvn_client/merge_processor.c
URL: http://svn.apache.org/viewvc/subversion/branches/apply-processor/subversion/libsvn_client/merge_processor.c?rev=1922161&r1=1922160&r2=1922161&view=diff
==============================================================================
--- subversion/branches/apply-processor/subversion/libsvn_client/merge_processor.c (original)
+++ subversion/branches/apply-processor/subversion/libsvn_client/merge_processor.c Wed Nov 27 13:05:03 2024
@@ -616,11 +616,11 @@ record_tree_conflict(merge_apply_process
 #if TODO_FILTER_MERGEINFO
       if (HONOR_MERGEINFO(merge_b) && merge_b->merge_source.ancestral)
         {
-          struct svn_client__merge_source_t *source;
+          svn_client__merge_source_t *source;
           svn_client__pathrev_t *loc1;
           svn_client__pathrev_t *loc2;
-          svn_merge_range_t range =
-            {SVN_INVALID_REVNUM, SVN_INVALID_REVNUM, TRUE};
+          svn_revnum_t start_rev;
+          svn_revnum_t end_rev;
 
           /* We are honoring mergeinfo so do not blindly record
            * a conflict describing the merge of
@@ -628,25 +628,27 @@ record_tree_conflict(merge_apply_process
            * SOURCE->LOC2->URL@SOURCE->LOC2->REV
            * but figure out the actual revision range merged. */
           (void)find_nearest_ancestor_with_intersecting_ranges(
-            &(range.start), &(range.end),
+            &start_rev, &end_rev,
             merge_b->children_with_mergeinfo,
             action != svn_wc_conflict_action_delete,
             local_abspath);
+
           loc1 = svn_client__pathrev_dup(merge_b->merge_source.loc1,
                                          scratch_pool);
           loc2 = svn_client__pathrev_dup(merge_b->merge_source.loc2,
                                          scratch_pool);
-          loc1->rev = range.start;
-          loc2->rev = range.end;
+          loc1->rev = start_rev;
+          loc2->rev = end_rev;
           source = svn_client__merge_source_create(loc1, loc2,
                                                    merge_b->merge_source.ancestral,
                                                    scratch_pool);
+
           SVN_ERR(make_conflict_versions(&left, &right, local_abspath,
                                          merge_left_node_kind,
                                          merge_right_node_kind,
                                          source, merge_b->target,
                                          result_pool, scratch_pool));
-        }
+      }
       else
 #endif
         SVN_ERR(make_conflict_versions(&left, &right, local_abspath,
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.