1132 replies to this topic

[Caught]

Ooit is me gevraagd om de versie weg te laten, dus vandaar.

Maar je hebt wel een punt. 

[Rob van der Does]

Allemaal terug te voeren op de problemen met het bouwen vanuit de bron (github). Die moet zorgen voor versie, update en de-installeren.

[Pedro_Newbie]

Je zou misschien als versienummer het commit nummer in github kunnen gebruiken, doe ik ook in de backupsuite.

Alleen wat is nu de ultieme bron, is dit github of de updates van Caught

[Loui (PD1LOI)]

Ik heb vanmorgen Caught ongeveer uitgelegd hoe github werkt,hij wil ermee aan de gang gaan maar ik weet niet of hij schrijfrechten heeft.als iemand van pli (Tech) dat even kan controleren? Anders komt het nooit goed.

Edited by Loui (PD1LOI), 23 October 2016 - 15:09.

[WanWizard]

Er is een invitation onderweg naar Caught.

[Loui (PD1LOI)]

Er is een invitation onderweg naar Caught.

Top dan kan ik hem verder helpen indien nodig.

Edited by Loui (PD1LOI), 23 October 2016 - 19:23.

[Caught]

Alles gaat goed t/m Commit en Push daarna krijg  ik Not all refs have bin pushed, lol   

Ik laat het nu even voor wat het is en kijk er later voor.

I.i.g. een nieuwe update van HetWeer v3.7

Changelog:

-Remove update from plugin

-Fix voor de radarscreens in OpenAtV

-De ipk file is ook een stuk kleiner geworden.

Attached Files

•  enigma2-plugin-extensions-hetweer_3.7_all.ipk   180.59KB   16 downloads

Edited by Caught, 23 October 2016 - 20:45.

[Rob van der Does]

Alles gaat goed t/m Commit en Push daarna krijg  ik Not all refs have bin pushed, lol  

Dan heb je waarschijnlijk geen 'pull' gedaan vlak voordat je pushte. Dat kan je alsnog doen.

[Pedro_Newbie]

Ik krijg een crash op het tekstbestand Background.txt
 

< 44931.833> Traceback (most recent call last):
< 44931.833>   File "/usr/lib/enigma2/python/Components/ActionMap.py", line 46, in action
< 44931.834>     res = self.actions[action]()
< 44931.834>   File "/usr/lib/enigma2/python/Screens/PluginBrowser.py", line 133, in save
< 44931.834>     self.run()
< 44931.834>   File "/usr/lib/enigma2/python/Screens/PluginBrowser.py", line 137, in run
< 44931.835>     plugin(session=self.session)
< 44931.835>   File "/usr/lib/enigma2/python/Plugins/Extensions/HetWeer/plugin.py", line 977, in main
< 44931.837>     with open('/usr/lib/enigma2/python/Plugins/Extensions/HetWeer/Images/background.txt', 'rb') as f:
< 44931.838> IOError: [Errno 2] No such file or directory: '/usr/lib/enigma2/python/Plugins/Extensions/HetWeer/Images/background.txt'

 
 
Regel 977 komt niet overeen met het origineel omdat ik wat lines heb toegevoegd, namelijk het automatisch bijhouden van het versienummer (gelijk aan de laatste commit)
 
 
Deze code

with open("/var/lib/opkg/info/enigma2-plugin-extensions-hetweer.control") as origin:
    for versie in origin:
        if not "Version: " in versie:
            continue
        try:
            versienummer = versie.split('+')[1]
        except IndexError:
            print

en dan toevoegen aan de omschrijving zoals dit

def Plugins(path, **kwargs):
    global plugin_path
    plugin_path = path
    return PluginDescriptor(name="HetWeer", description="BuienRadar & WeerInfo, versie " + versienummer,
                            icon="Images/weerinfo.png",
                            where=[PluginDescriptor.WHERE_EXTENSIONSMENU, PluginDescriptor.WHERE_PLUGINMENU], fnc=main)

[theparasol]

Ik vind het checken op aanwezigheid van bepaalde skin niet echt handig.

 

Kun je het niet beter zo doen (zoals in openwebif):

 

if not type == "Weerbericht":
                distro = "unknown"
                try:
                    f = open("/etc/opkg/all-feed.conf",'r')
                    oeline = f.readline().strip().lower()
                    f.close()
                    distro = oeline.split( )[1].replace("-all","")
                except:
                    pass
                if distro == "openpli":
                    self.session.open(radarScreenop)
                else:
                    self.session.open(radarScreenoatv)

[Caught]

Hey Pedro de background.txt is niet op Github geraakt waardoor deze ook gemist wordt. (mijn fout).

Ofwel kan je hem zelf aanmaken in de Image folder ofwel moet je de 3.7 IPK installeren.

Indien je hem zelf aanmaakt moet je er het getal 10 in plaatsen.

 

@ Rob en Loui ik kijk later voor Github alvast bedankt voor de begeleiding    

[Caught]

 

Ik vind het checken op aanwezigheid van bepaalde skin niet echt handig.

 

Kun je het niet beter zo doen (zoals in openwebif):

if not type == "Weerbericht":
                distro = "unknown"
                try:
                    f = open("/etc/opkg/all-feed.conf",'r')
                    oeline = f.readline().strip().lower()
                    f.close()
                    distro = oeline.split( )[1].replace("-all","")
                except:
                    pass
                if distro == "openpli":
                    self.session.open(radarScreenop)
                else:
                    self.session.open(radarScreenoatv)

Ik ga het bekijken, ik vond de check op MetrixHD ook niet 100% maar voorlopig ok. Bedankt voor het vb!!

[Caught]

 
 
Regel 977 komt niet overeen met het origineel omdat ik wat lines heb toegevoegd, namelijk het automatisch bijhouden van het versienummer (gelijk aan de laatste commit)
 
 
Deze code

with open("/var/lib/opkg/info/enigma2-plugin-extensions-hetweer.control") as origin:
    for versie in origin:
        if not "Version: " in versie:
            continue
        try:
            versienummer = versie.split('+')[1]
        except IndexError:
            print

en dan toevoegen aan de omschrijving zoals dit

def Plugins(path, **kwargs):
    global plugin_path
    plugin_path = path
    return PluginDescriptor(name="HetWeer", description="BuienRadar & WeerInfo, versie " + versienummer,
                            icon="Images/weerinfo.png",
                            where=[PluginDescriptor.WHERE_EXTENSIONSMENU, PluginDescriptor.WHERE_PLUGINMENU], fnc=main)

 

Pedro, dit zal ik ook toevoegen, dan wordt het toch iets makkelijker om bij te houden.

Grts. en thx

[Pedro_Newbie]

Met commit 181 is er e.e.a. niet goed gegaan denk ik, bij de temperaturen staat er nu °in plaats van alleen het graden symbool °

[Caught]

Met commit 181 is er e.e.a. niet goed gegaan denk ik, bij de temperaturen staat er nu °in plaats van alleen het graden symbool °

Ik heb het gezien, normaal gezien zou dit morgen goed moeten komen.

Ondertussen kan je best de IPK file installeren.

Attached Files

•  enigma2-plugin-extensions-hetweer_3.7_all.ipk   180.59KB   10 downloads

[Pedro_Newbie]

Ik kan er mee leven maar ik dacht ik zal het even melden.
Het door elkaar gebruiken van losse IPK's en downloads van de feed via Github is niet aan te raden, de versienummers gaan dan hopeloos door elkaar heen lopen

[Loui (PD1LOI)]

 

Met commit 181 is er e.e.a. niet goed gegaan denk ik, bij de temperaturen staat er nu °in plaats van alleen het graden symbool °

Ik heb het gezien, normaal gezien zou dit morgen goed moeten komen.

Ondertussen kan je best de IPK file installeren.

 

Ja ik heb hier een vraagje over, in het begin had je de versie links boven in de skin staan (HetWeer versie 1.5 ) als voorbeeld, kan dat nu ook nog maar dan met de huidige versie?

[Caught]

Ja dat kan, maar dit komt niet overeen met github.

[WanWizard]

Je kunt in git een versie 'taggen', hier kun je je versie nummer kwijt. Dan kun je ook via github zo de source van een bepaalde versie terughalen.

[Happysat]

Alles gaat goed t/m Commit en Push daarna krijg  ik Not all refs have bin pushed, lol   

Ik laat het nu even voor wat het is en kijk er later voor.

I.i.g. een nieuwe update van HetWeer v3.7

Changelog:

-Remove update from plugin

-Fix voor de radarscreens in OpenAtV

-De ipk file is ook een stuk kleiner geworden.

Werkt nu prima in ATV top werk!