UniversalDevicesInc / Polyglot

Polyglot Open Source
MIT License
24 stars 2 forks source link

Merge-to-mainline, build, and release process needs to be defined #27

Open mjwesterhof opened 8 years ago

mjwesterhof commented 8 years ago

We need a documented process and/or checklist and/or Makefile rules to help perform all the necessary actions required to merge changes into the main branch, build on the Pi, Mac, and PC platforms, commit the results, and tag that version on the mainline.

Let the discussion commence!

Einstein42 commented 8 years ago

I say we create an 'unstable-rc' branch. Once we have added a feature or fixed something we merge to unstable-rc... all the developers should be running unstable-rc... we put a change freeze on on unstable-rc after the merge is performed for 2 days. If no issues arrise we can build and push to unstable-release and reopen unstable-rc for new features/fixes.

mkohanim commented 8 years ago

Hi James, That’s a great idea!

With kind regards,


Michel Kohanim CEO

(p) 818.631.0333 (f) 818.436.0702 http://www.universal-devices.comhttp://www.universal-devices.com/


From: James [mailto:notifications@github.com] Sent: Monday, April 4, 2016 8:31 AM To: UniversalDevicesInc/Polyglot Polyglot@noreply.github.com Subject: Re: [UniversalDevicesInc/Polyglot] Merge-to-mainline, build, and release process needs to be defined (#27)

I say we create an 'unstable-rc' branch. Once we have added a feature or fixed something we merge to unstable-rc... all the developers should be running unstable-rc... we put a change freeze on on unstable-rc after the merge is performed for 2 days. If no issues arrise we can build and push to unstable-release and reopen unstable-rc for new features/fixes.

— You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHubhttps://github.com/UniversalDevicesInc/Polyglot/issues/27#issuecomment-205352011

Einstein42 commented 8 years ago

https://github.com/UniversalDevicesInc/Polyglot/wiki/Build-and-Release-Information

Anything else you guys want on that?

mkohanim commented 8 years ago

Hi James,

Thank you. This is excellent. I am sending an email blast. With regards to the documentation, I shall have it pointed to: https://github.com/UniversalDevicesInc/Polyglot/blob/unstable-release/README.md … is that correct?

With kind regards,


Michel Kohanim CEO

(p) 818.631.0333 (f) 818.436.0702 http://www.universal-devices.comhttp://www.universal-devices.com/


From: James [mailto:notifications@github.com] Sent: Monday, April 4, 2016 9:15 AM To: UniversalDevicesInc/Polyglot Polyglot@noreply.github.com Cc: Michel Kohanim michel@universal-devices.com Subject: Re: [UniversalDevicesInc/Polyglot] Merge-to-mainline, build, and release process needs to be defined (#27)

https://github.com/UniversalDevicesInc/Polyglot/wiki/Build-and-Release-Information

Anything else you guys want on that?

— You are receiving this because you commented. Reply to this email directly or view it on GitHubhttps://github.com/UniversalDevicesInc/Polyglot/issues/27#issuecomment-205371969

Einstein42 commented 8 years ago

Yes sir.

mkohanim commented 8 years ago

Thank you.

With kind regards,


Michel Kohanim CEO

(p) 818.631.0333 (f) 818.436.0702 http://www.universal-devices.comhttp://www.universal-devices.com/


From: James [mailto:notifications@github.com] Sent: Monday, April 4, 2016 9:32 AM To: UniversalDevicesInc/Polyglot Polyglot@noreply.github.com Cc: Michel Kohanim michel@universal-devices.com Subject: Re: [UniversalDevicesInc/Polyglot] Merge-to-mainline, build, and release process needs to be defined (#27)

Yes sir.

— You are receiving this because you commented. Reply to this email directly or view it on GitHubhttps://github.com/UniversalDevicesInc/Polyglot/issues/27#issuecomment-205380952

mjwesterhof commented 8 years ago

(Apologize if this is already documented; I'm at work and can't review in depth until this evening) -- we should make sure we increment the version number in version.py at the correct times, so that we know what the user is running from the logs... otherwise this could get really confusing fast!

Last I checked, the development head was at 0.0.1 - the version number is a string so we can easily put "rc", or "beta" suffixes on if desired. It's also in a file all by itself so the inevitable manual merges we'll get when merging git branches will end up isolated and easy to resolve.

Einstein42 commented 8 years ago

Let me understand mjwesterhof. You modified everything to use the PGVERSION imported from version.py so we only need to update it there correct?

Edit I checked he did. That's great. Thanks for doing that. We will increment a single number in the third octet per merge to unstable-rc. That work for everyone?

mjwesterhof commented 8 years ago

Works for me!

Einstein42 commented 8 years ago

Ok. This is now in effect! Go team.

mjwesterhof commented 8 years ago

@mkohanim - Can you create a repository alongside this one for the binaries and documentation? This would allow us to remove the generated artifacts from the source repo, and then we'd simply check them in and tag them in the binary repo.

Users would never need to check out or deal with the source repo; they'd only ever reference the release repo for the docs and pyz files.

Thoughts and comments from the team welcomed!

mkohanim commented 8 years ago

@mjweterhof, of course. Do you mean something like UniversalDevicesInc/Polyglot-Bin?

With kind regards, Michel

Einstein42 commented 8 years ago

I think that is fine, or even 'Polyglot-Release'

mkohanim commented 8 years ago

Done! Thank you.

jimboca commented 8 years ago

@mjwesterhof Can we merge the current development branch up the chain? I would like to release my camera server and don't want to ask everyone to grab the development branch?

Einstein42 commented 8 years ago

@jimboca go ahead and pull dev into unstable-rc please

jimboca commented 8 years ago

done