Hi all,

Just a question from the sidelines...

Where does anything have to be merged at all?

Usually the release branches are simply dead ends and they aren't merged back 
to develop. Some projects merge the release tags to master, but noone merges 
the release branch back to develop. That would cause a mess anyway.

So I don't quite understand where there are merge conflicts.

Chris
________________________________
Von: Carlos Rovira <[email protected]>
Gesendet: Dienstag, 21. April 2020 15:25
An: Apache Royale Development <[email protected]>
Betreff: Re: [royale-compiler] branch release/0.9.7 updated: Update compiler 
build tools version

Hi Alex,

sorry to reach late to this. From other email from you what I understand
was that we should continue our work in develop branch, while release was
already cut and the commits that RM want to cherry pick was up to them. So
I was acting with that in mind.

Benign said that, merge conflicts can always happen at any time when
merging. I was just recommending in that particular one to cherry pick,
since that ensures no merge conflicts, but is clear that in other parts, we
can be making more changes that conflict with actual release work and that
needs to solved manually, but this is normal.

thanks




El lun., 20 abr. 2020 a las 17:32, Alex Harui (<[email protected]>)
escribió:

> IMO, dev branch should not have been updated as it did not need the code
> change in 1.2.0.  Cherry picking can also cause merge conflicts.
>
> But you can't eliminate the risks of merge conflicts.  So just keep on
> going with what is committed currently.
>
> -Alex
>
> On 4/20/20, 4:40 AM, "Yishay Weiss" <[email protected]> wrote:
>
>     Carlos, are you sure your commit belongs in dev at this stage? If we
> are going to merge release into dev anyway, maybe it’s best to wait until
> then to avoid this conflict. Also, I’m not a Git expert, but wouldn’t the
> merge algorithm detect the identity in content anyway and avoid conflict?
>
>     From: Piotr Zarzycki<mailto:[email protected]>
>     Sent: Monday, April 20, 2020 2:30 PM
>     To: Apache Royale Development<mailto:[email protected]>
>     Subject: Re: [royale-compiler] branch release/0.9.7 updated: Update
> compiler build tools version
>
>     Agree with Carlos - It may end up with problems. Actually after release
>     branch should be probably just merged to develop and probably master
> (not
>     sure here) without changes.
>
>     pon., 20 kwi 2020 o 13:27 Carlos Rovira <[email protected]>
>     napisał(a):
>
>     > Hi Yishay,
>     >
>     > since I did the same in develop, will not be better to just cherry
> pick my
>     > commit on release branch?
>     > I think doing this way will make the merge to develop from release
>     > problematic since will find different changes (although are the
> same) and
>     > you'll need to resolve the merge manually.
>     >
>     > maybe I'm wrong
>     >
>     > Thanks
>     >
>     >
>     > El lun., 20 abr. 2020 a las 12:58, <[email protected]> escribió:
>     >
>     > > This is an automated email from the ASF dual-hosted git repository.
>     > >
>     > > yishayw pushed a commit to branch release/0.9.7
>     > > in repository
> https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgitbox.apache.org%2Frepos%2Fasf%2Froyale-compiler.git&amp;data=02%7C01%7Caharui%40adobe.com%7C0126494d6a6e42a426a708d7e51fab8e%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C0%7C637229796541721500&amp;sdata=q6T8B558JaACJJ7qSzbhsV1c4eBL774lU8%2FSuUmegu4%3D&amp;reserved=0
>     > >
>     > >
>     > > The following commit(s) were added to refs/heads/release/0.9.7 by
> this
>     > > push:
>     > >      new 1b8f24a  Update compiler build tools version
>     > > 1b8f24a is described below
>     > >
>     > > commit 1b8f24aee6bf21d25ae75b039b88c00a596c46bb
>     > > Author: DESKTOP-RH4S838\Yishay <[email protected]>
>     > > AuthorDate: Mon Apr 20 13:58:20 2020 +0300
>     > >
>     > >     Update compiler build tools version
>     > > ---
>     > >  pom.xml | 2 +-
>     > >  1 file changed, 1 insertion(+), 1 deletion(-)
>     > >
>     > > diff --git a/pom.xml b/pom.xml
>     > > index 3fea91d..1ee3dd9 100644
>     > > --- a/pom.xml
>     > > +++ b/pom.xml
>     > > @@ -47,7 +47,7 @@
>     > >
>     > >
>     >
> <project.reporting.outputencoding>UTF-8</project.reporting.outputencoding>
>     > >      <maven.version>3.3.1</maven.version>
>     > >
>     > > -
> <compiler-build-tools.version>1.1.0</compiler-build-tools.version>
>     > > +
> <compiler-build-tools.version>1.2.0</compiler-build-tools.version>
>     > >
> <compiler-jburg-types.version>1.1.0</compiler-jburg-types.version>
>     > >
>     > >      <flex.version>4.15.0</flex.version>
>     > >
>     > >
>     >
>     > --
>     > Carlos Rovira
>     >
> https://nam04.safelinks.protection.outlook.com/?url=http%3A%2F%2Fabout.me%2Fcarlosrovira&amp;data=02%7C01%7Caharui%40adobe.com%7C0126494d6a6e42a426a708d7e51fab8e%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C0%7C637229796541731496&amp;sdata=oGWnWGINE4gGtezx2MeuZvo3cqc5CSvcx7AdrYJeWV8%3D&amp;reserved=0
>     >
>
>
>     --
>
>     Piotr Zarzycki
>
>     Patreon: *
> https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fwww.patreon.com%2Fpiotrzarzycki&amp;data=02%7C01%7Caharui%40adobe.com%7C0126494d6a6e42a426a708d7e51fab8e%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C0%7C637229796541731496&amp;sdata=C79G11fhsKtkw5%2BaffnMEjyUn0vBSQol%2B0b7UJjvatw%3D&amp;reserved=0
>     <
> https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fwww.patreon.com%2Fpiotrzarzycki&amp;data=02%7C01%7Caharui%40adobe.com%7C0126494d6a6e42a426a708d7e51fab8e%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C0%7C637229796541731496&amp;sdata=C79G11fhsKtkw5%2BaffnMEjyUn0vBSQol%2B0b7UJjvatw%3D&amp;reserved=0
> >*
>
>
>
>

--
Carlos Rovira
http://about.me/carlosrovira

Reply via email to