Cruisers Forum
 


Join CruisersForum Today

Reply
 
Thread Tools Rate Thread Display Modes
Old 19-01-2015, 18:13   #1
Registered User

Join Date: May 2013
Location: NSW, Australia
Boat: Richter 42
Posts: 883
Unfreeze changes post 4.0.0

Hi,
Do you know when the freeze period for OCPN 4.0.0 will end? I have some changes I would like to get into the next beta stream and some of them are getting a bit old now.

Thanks
Jon
__________________

__________________
jongough is offline   Reply With Quote
Old 19-01-2015, 18:42   #2
Registered User

Join Date: Feb 2010
Location: On the go. Not in Prague.
Posts: 4,016
Re: Unfreeze changes post 4.0.0

Jon...
It is over, just post pull requests into the master, the stuff qualifying also as a 4.0.x fix should go to v4.0.x branch in separate pull request or at least should be very clearly marked as a candidate for backporting.
I just don't want to start merging the stuff until Dave returns from his well deserved vacation... But be warned, as I know him, the vacation will finish with a huge patch applied to the master as usually

Pavel
__________________

__________________
nohal is offline   Reply With Quote
Old 19-01-2015, 20:50   #3
Marine Service Provider
 
bdbcat's Avatar

Join Date: Mar 2008
Posts: 4,877
Re: Unfreeze changes post 4.0.0

Jon/Pavel...

I'm listening, but not too closely.

No problem with merging pull requests into master whenever you think they are ready. I understand the staleness factor...

We should discuss what gets backported as it comes up. I don't see anything that is a critical backport candidate yet....


Are you expecting a big patch from me when when I re-surface? Well, maybe.....

Thanks
Dave
__________________
bdbcat is offline   Reply With Quote
Old 19-01-2015, 22:27   #4
Registered User

Join Date: Feb 2010
Location: On the go. Not in Prague.
Posts: 4,016
Re: Unfreeze changes post 4.0.0

OK, I will start with it, hopefully tomorrow, to move the baseline and minimize the number of merging conflicts, which are already there now - don't expect any lightning speed though...

Pavel
__________________
nohal is offline   Reply With Quote
Old 20-01-2015, 12:58   #5
Registered User

Join Date: May 2013
Location: NSW, Australia
Boat: Richter 42
Posts: 883
Re: Unfreeze changes post 4.0.0

Great! Some of the changes I have put in are being tested/used from windows installs I put up on drop box, but they are getting a bit stale now as well. I am happy to update the windows installs, but I would prefer that the changes get incorporated, particularly the 'boundary' one as it is big, before too many changes happen.

Thanks
Jon
__________________
jongough is offline   Reply With Quote
Old 20-01-2015, 19:26   #6
Registered User

Join Date: May 2013
Location: NSW, Australia
Boat: Richter 42
Posts: 883
Re: Unfreeze changes post 4.0.0

Pavel,
I have just noticed a possible clash with your FS#1670 change and my FS#1533 boundary change. The boundary change makes quite a few modifications to chcanv.cpp and it may be easier to fit your change into my version of this file (there may be some extra areas you have to make the changes to) than me try and rework and add to your change. Is that possible?

Thanks
Jon
__________________
jongough is offline   Reply With Quote
Old 20-01-2015, 19:37   #7
Registered User

Join Date: Feb 2010
Location: On the go. Not in Prague.
Posts: 4,016
Re: Unfreeze changes post 4.0.0

Jon...
Yes, I will first merge the pull requests and then forward-port my fixes from the v4.0.x branch - otherwise I would probably get crazy...

Thanks for cross-checking

Pavel
__________________
nohal is offline   Reply With Quote
Old 20-01-2015, 20:07   #8
Registered User

Join Date: Feb 2010
Location: On the go. Not in Prague.
Posts: 4,016
Re: Unfreeze changes post 4.0.0

Jon...
Please, in the future try to use git rebase on upstream/master instead of git merge with upstream/master in the feature branches - it will avoid the "Merge remote-tracking branch 'upstream/master'" commits.
And I was wondering why pulling your repo took so loooong and stepped on https://github.com/jongough/OpenCPN/...aster/test-win - not exactly a good idea

Pavel
__________________
nohal is offline   Reply With Quote
Old 21-01-2015, 13:56   #9
Registered User

Join Date: May 2013
Location: NSW, Australia
Boat: Richter 42
Posts: 883
Re: Unfreeze changes post 4.0.0

I will try the rebase thing with git. But being new to git I am using what seems to be working and slowly working out what is best. I am using SmartGit on my PC and one of the updates seems to have turned on a facility I was not expecting and some stuff got uploaded that was not supposed to go. I ment to delete test-win directory last night, but I was only in the office for about 15 mins and did the OpenGL fix then forgot to clean up git. Sorry about that. I will try and remember tonight. I don't know how to delete stuff using the web interface or I could do it now.

So in future I should do a
git pull upstream
then a
git rebase upstream/master
rather than a merge? Do I do this to my working branch or to the master branch?
__________________
jongough is offline   Reply With Quote
Old 21-01-2015, 14:24   #10
Registered User

Join Date: Feb 2010
Location: On the go. Not in Prague.
Posts: 4,016
Re: Unfreeze changes post 4.0.0

Quote:
Originally Posted by jongough View Post
So in future I should do a
git pull upstream
then a
git rebase upstream/master
rather than a merge? Do I do this to my working branch or to the master branch?
To any branch you want to bring up to date, typically to your working branch, at least before creating a pull request (but of course you usually do it every now and then when you see changes in the upstream master).

Pavel
__________________

__________________
nohal is offline   Reply With Quote
Reply

Thread Tools
Display Modes Rate This Thread
Rate This Thread:

Posting Rules
You may not post new threads
You may not post replies
You may not post attachments
You may not edit your posts

BB code is On
Smilies are On
[IMG] code is On
HTML code is Off
Trackbacks are Off
Pingbacks are Off
Refbacks are Off




Copyright 2002- Social Knowledge, LLC All Rights Reserved.

All times are GMT -7. The time now is 11:45.


Google+
Powered by vBulletin® Version 3.8.8 Beta 1
Copyright ©2000 - 2017, vBulletin Solutions, Inc.
Social Knowledge Networks
Powered by vBulletin® Version 3.8.8 Beta 1
Copyright ©2000 - 2017, vBulletin Solutions, Inc.

ShowCase vBulletin Plugins by Drive Thru Online, Inc.