#4738 closed enhancement (fixed)
Enable/Disable ecm.info file
| Reported by: | luckysquid | Owned by: | nautilus7 |
|---|---|---|---|
| Priority: | minor | Component: | DVBApi |
| Severity: | low | Keywords: | ecm.info |
| Cc: | Sensitive: | no |
Description
Reason for enhancement
Enable/Disable? ecm.info file
ecminfo_file = 0|1
ecm.info file:
0 = Disable ecm.info file
1 = Enable ecm.info file (default)
attached the diff files. (code and docs).
diff from last rev 11537
all code are tested, please implemented into trunk
Possible impacts on other features
none
Attachments (11)
Change History (19)
Changed 6 days ago by
| Attachment: | oscam.conf.txt.patch added |
|---|
Changed 6 days ago by
| Attachment: | oscam.conf.5.patch added |
|---|
Changed 6 days ago by
| Attachment: | oscam.conf.5.html.patch added |
|---|
Changed 6 days ago by
| Attachment: | module-dvbapi.c.patch added |
|---|
Changed 6 days ago by
| Attachment: | module-dvbapi-mca.c.patch added |
|---|
Changed 6 days ago by
| Attachment: | module-dvbapi-azbox.c.patch added |
|---|
Changed 6 days ago by
| Attachment: | globals.h.patch added |
|---|
Changed 6 days ago by
| Attachment: | oscam-config-global.c.patch added |
|---|
Changed 6 days ago by
| Attachment: | module-webif.c.patch added |
|---|
Changed 6 days ago by
| Attachment: | dvbapi.html.patch added |
|---|
comment:1 Changed 5 days ago by
comment:2 Changed 5 days ago by
@tom001x: thanks for patch
@Opti: exactly, most skin use this file and sometimes it is ugly to see it when zapping channel. Of course default option is with ecm.info.
I' am oscam and streamboard fun for over 10 years, it's a pleasure to contribute to the project.
Great team!
comment:3 Changed 4 days ago by
@Opti: I tried to apply entire patch posted by you and I don't notice any problem. I think that you can apply this to trunk.
Best regards!
Changed 3 days ago by
| Attachment: | ecm_info_on_off.patch added |
|---|
comment:4 Changed 3 days ago by
then please the replaced Version with clean -->module-webif.c
Best regards! & @all thanks for patch
comment:6 Changed 45 hours ago by
| Owner: | set to nautilus7 |
|---|---|
| Status: | new → assigned |
I don't see the reason why you need to disable the ecm.info file, but I don't see a problem having an option to enable/disable it either.
So, are all the separate patches included in the patch Opti posted? I will test it myself and commit it, probably tomorrow.
comment:7 Changed 30 hours ago by
| Resolution: | → fixed |
|---|---|
| Status: | assigned → closed |
Implemented in revision 11539.
What's the disables good for?
Most SKIN's use this info.