1132 replies to this topic

[Loui (PD1LOI)]

Het enige wat ik gedaan heb is de plugin in de juiste volgorde zetten, als jr deze via applicaties download dan komt deze op de juiste plek in je box. En naam veranderen van OpenEnigma2/HetWeer moet hernoemd worden volgens mij.

Verstuurd vanaf mijn One X met Tapatalk

[Rob van der Does]

Kijk maar even naar mijn pullrequest: volgens mij is het nu allemaal in orde.

[Loui (PD1LOI)]

Ik heb

 

Kijk maar even naar mijn pullrequest: volgens mij is het nu allemaal in orde.

Alles is nu goed daar, maar er is een probleem die door Caught moet worden aangepast of door ons, Namelijk in de plugin wordt de pad naar Weerinfo gegeven en niet naar HetWeer dus zie je als je op weerinfo drukt geen achtergond alleen de text.

Hier een voorbeeld:<ePixmap pixmap="/usr/lib/enigma2/python/Plugins/Extensions/WeerInfo/Images/buttons/red26.png

[WanWizard]

Waarom zo moeilijk doen, en niet gewoon een nieuwe repo maken voor "Weerinfo", en die andere verwijderen?

 

Ik zie trouwens dat er een "upstream" branch is die veel recenter is dan "master", https://github.com/E...ommits/upstream, en waar HetWeer wel in staat.

 

Ik zie nu ook dat OpenPLi blijkbaar niet bij is, de repo staat gepint op de commit van 16 april, dus de autoshutdown fixes worden niet meegenomen. Handig.

[Rob van der Does]

Ik heb
 

Kijk maar even naar mijn pullrequest: volgens mij is het nu allemaal in orde.

Alles is nu goed daar, maar er is een probleem die door Caught moet worden aangepast of door ons, Namelijk in de plugin wordt de pad naar Weerinfo gegeven en niet naar HetWeer dus zie je als je op weerinfo drukt geen achtergond alleen de text.
Hier een voorbeeld:<ePixmap pixmap="/usr/lib/enigma2/python/Plugins/Extensions/WeerInfo/Images/buttons/red26.png

Dat heb ik toch net allemaal aangepast? Ik kan geen fout meer ontdekken. Heb je mijn pullrequest bekeken?

[Erik Slagter]

Goed bezig mannen, we zijn er nog niet helemaal kennelijk, even doorbijten En dan hoeft er nooit meer met ipk's rondgezeuld te worden!

[Rob van der Does]

Waarom zo moeilijk doen, en niet gewoon een nieuwe repo maken voor "Weerinfo", en die andere verwijderen?

Op verzoek van Erik, en helemaal niet moeilijk

Ik zie trouwens dat er een "upstream" branch is die veel recenter is dan "master", https://github.com/E...ommits/upstream, en waar HetWeer wel in staat.

Daar weet ik niets van, maar PLi heeft vannacht gewoon de aangapaste plugin 'HetWeer' gebouwd vanuit https://github.com/E...nplugin-HetWeer

Maar ik heb nog steeds geen antwoord gehad op

Wat ik alleen niet begrijp, is dat de plugin 'HetWeer' niet voorkomt in https://github.com/E...s/e2openplugins maar wel in https://github.com/E2OpenPlugins

[Rob van der Does]

Ik heb
 

Kijk maar even naar mijn pullrequest: volgens mij is het nu allemaal in orde.

Alles is nu goed daar, maar er is een probleem die door Caught moet worden aangepast of door ons, Namelijk in de plugin wordt de pad naar Weerinfo gegeven en niet naar HetWeer dus zie je als je op weerinfo drukt geen achtergond alleen de text.
Hier een voorbeeld:<ePixmap pixmap="/usr/lib/enigma2/python/Plugins/Extensions/WeerInfo/Images/buttons/red26.png

Dat heb ik toch net allemaal aangepast? Ik kan geen fout meer ontdekken. Heb je mijn pullrequest bekeken?

Je moet alleen wel even dat request mergen, want ik heb daar geen schrijfrechten.

[Loui (PD1LOI)]

Is al gebeurd Rob!

[WanWizard]

Maar ik heb nog steeds geen antwoord gehad op

Wat ik alleen niet begrijp, is dat de plugin 'HetWeer' niet voorkomt in https://github.com/E...s/e2openplugins maar wel in https://github.com/E2OpenPlugins

 

Dat heb je wel.

 

Ik heb je verteld dat er twee branches in die repo zitten, master en upstream, en die laatste is veel recenter dan die eerste. En ik heb ook gezegd dat OpenPLi van geen van beide branches de HEAD gebruikt, maar de repo heeft vast gepint op een commit hash, in dit geval die van April in de upstream branch.

 

En dat is waarom HetWeer gewoon gebouwd wordt, want die zit daar gewoon in.

[Rob van der Does]

Zoals ik al zei: 'HetWeer' is gisteren aangepast in de branch die ik liet zien, en die versie was vanochtend beschikbaar op de feeds.
En de enige versie die ik ken, is die waar de homepage van PLi naar verwijst; die is dan ook gebruikt.

[WanWizard]

Ja, en? Wat heeft dat te maken met je vorige vraag en mijn antwoord daar op?

 

De versie die gebouwd wordt heeft niks te maken met hoe https://github.com/E...s/e2openplugins er uit ziet en hoe deze gebruikt wordt. Enige relevante daar is dat het recipe gebruikt wordt, en dat is het geval.

[Rob van der Does]

OK, ik begrijp het antwoord dus niet.

[WanWizard]

OpenPLi heeft de repository gepint op een specifieke commit hash: https://github.com/O...recipes-openpli, in dit geval d8f3cecf58e7c3013eec77dea714db8e59e39ff4.

 

Deze hash is een commit in de upstream branch: https://github.com/E...714db8e59e39ff4. Hier staat de recipe van de HetWeer plugin gewoon tussen: https://github.com/E...714db8e59e39ff4 en dus komt deze zonder problemen in de image terecht. In de recipe van de plugin zelf is geen sprake van een versie: https://github.com/E...ions-hetweer.bb dus daar wordt gewoon de laatste stand van zaken gepakt.

[Rob van der Does]

OK, gevonden, duidelijk, dank.
Soms snap je het pas als je het door hebt...

@Caught:
nog te doen:
1- plugin update volledig verwijderen.
2- plugin versie uit de py verwijderen.

[Loui (PD1LOI)]

Heerlijk is dat toch he! Niks geen gezijk gewoon kijken en meewerken Toppie WanWizard.

[Caught]

Ok bedankt aan allen die het werk opgeknapt hebben

 

Ik heb hier de aanpassing van v 2.3 dus zonder het update verhaal.

En de versienrs. zijn eruit genomen, ondertussen heb ik nog een ander NA (not available ) icoontje toegevoegd.  

Kan ik hier de nieuwe ingepakte files zomaar "dumpen"? Thanks bij voorbaat.   

Attached Files

•  WeerInfo2.3.rar   1.47MB   8 downloads

[Rob van der Does]

Hi, ik denk dat je de verkeerde file hebt gestuurd: dit is de oude versie (met alle paden naar WeerInfo i.p.v. HetWeer).

Edited by Rob van der Does, 13 September 2016 - 02:17.

[Loui (PD1LOI)]

Vanmorgen een update gedaan hier werkt nu alles prima.

[Loui (PD1LOI)]

Toch nog een minpunt, ik heb HetWeer verwijderd en daarna enigma herstart en gekeken of de map verwijderd was.

daarna heb ik via applicatie deplugin HetWeer(2.0+git133) geïnstalleerd maar nu zie ik dat de map Images niet word gedownload.

Deze heb ik handmatig toegevoegd op mijn box, Dus Rob er staat iets niet goed in de Github bouwserver.