Skip to content

LEAmDNS #5370

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 6 commits into from
Closed

LEAmDNS #5370

wants to merge 6 commits into from

Conversation

LaborEtArs
Copy link
Contributor

Created a new branch to solve commit conflicts with recent changes to ESP8266mDNSResponder.cpp by d-a-v.
Also removed millis() roll-over issues.

LaborEtArs and others added 5 commits November 24, 2018 09:00
Created a new branch to solve commit conflicts with recent changes to ESP8266mDNSResponder.cpp by d-a-v.
* PolledTimeout Class for wrapping millis() loops

* Add yield policies, improve reset, add host tests

* Fix copyright, comments

* adjust host tests for better time precision

* add fuzzyness to timing tests for CI jitter

* add blink example with polledTimeout

* improve namespace and type naming, add copyright, comments

* fix astyle
@d-a-v
Copy link
Collaborator

d-a-v commented Nov 26, 2018

@LaborEtArs
I locally merged your branch, but there is no way to make a simple / short PR on your branch (to my github knowledge).
The only conflict is library.properties and to solve it you just need to replace the file with your version.

To handle this from scratch, and because we can't do it from github (only from cmdline), you can try:

  • clone the official repo on some new directory
    git clone https://github.com/esp8266/Arduino.git
  • add your source
    cd Arduino
    git remote add lea https://github.com/LaborEtArs/Arduino.git
  • create a (local) new branch with the same name as yours
    git checkout -b LEAmDNS
  • This branch is even with current github versionof the core. Now merge it with your repository
    git pull lea LEAmDNS
  • You have the conflict, now edit libraries/ESP8266mDNS/library.properties, remove anything which is not yours
  • commit your changes
    git add libraries/ESP8266mDNS/library.properties
    git commit
  • leave the commit message (which is merge blah), save it
  • now you are even, push to your branch
    git push lea LEAmDNS

@LaborEtArs
Copy link
Contributor Author

Did all, what d-a-v recommended, now the PR seems to be mergeable... Anything left to do???

@d-a-v
Copy link
Collaborator

d-a-v commented Nov 26, 2018

Anything left to do???

Yes, update libraries/ESP8266mDNS/src/ESP8266mDNS.h L42:
libraries/ESP8266mDNS/src/ESP8266MDNS.h ->
libraries/ESP8266mDNS/src/ESP8266mDNS.h

@d-a-v
Copy link
Collaborator

d-a-v commented Nov 26, 2018

Here are your two examples formatted by astyle for CI:
https://gist.github.com/d-a-v/0b70725d46ac7314fc98e7b02538f678
You must replace yours by them for CI to pass.

@LaborEtArs
Copy link
Contributor Author

Updated 'libraries/ESP8266mDNS/src/ESP8266mDNS.h' and replaced 'libraries/ESP8266mDNS/examples/LEAmDNS/mDNS_Clock/mDNS_Clock.ino' and
'libraries/ESP8266mDNS/examples/LEAmDNS/mDNS_ServiceMonitor/mDNS_ServiceMonitor.ino'
then called
add libraries/ESP8266mDNS/src/ESP8266mDNS.h
add libraries/ESP8266mDNS/examples/LEAmDNS/mDNS_Clock/mDNS_Clock.ino
add libraries/ESP8266mDNS/examples/LEAmDNS/mDNS_ServiceMonitor/mDNS_ServiceMonitor.ino

and
git commit
but got
On branch LEAmDNS
nothing to commit, working tree clean

the final
git push lea LEAmDNS
produced
Everything up-to-date
I really don't like to walk around like a blind one...

@d-a-v
Copy link
Collaborator

d-a-v commented Nov 27, 2018

Your branch can be updated from github to synchronise with master.
Then you would need to git pull lea LEAmDNS

I reckon this can be tricky with big PR like yours and a master branch always updating.
If you are stuck, I can try to pull from your repository and propose PR on it.

@d-a-v d-a-v mentioned this pull request Nov 28, 2018
@devyte
Copy link
Collaborator

devyte commented Dec 5, 2018

Closing in favor of #5384 .

@devyte devyte closed this Dec 5, 2018
@d-a-v
Copy link
Collaborator

d-a-v commented Dec 5, 2018

closing in favour of #5442

@d-a-v d-a-v reopened this Dec 5, 2018
@d-a-v d-a-v closed this Dec 5, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants