#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)
by , 4 years ago
| Attachment: | oscam.conf.txt.patch added |
|---|
by , 4 years ago
| Attachment: | oscam.conf.5.patch added |
|---|
by , 4 years ago
| Attachment: | oscam.conf.5.html.patch added |
|---|
by , 4 years ago
| Attachment: | module-dvbapi.c.patch added |
|---|
by , 4 years ago
| Attachment: | module-dvbapi-mca.c.patch added |
|---|
by , 4 years ago
| Attachment: | module-dvbapi-azbox.c.patch added |
|---|
by , 4 years ago
| Attachment: | globals.h.patch added |
|---|
by , 4 years ago
| Attachment: | oscam-config-global.c.patch added |
|---|
by , 4 years ago
| Attachment: | module-webif.c.patch added |
|---|
by , 4 years ago
| Attachment: | dvbapi.html.patch added |
|---|
comment:1 by , 4 years ago
comment:2 by , 4 years ago
@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 by , 4 years ago
@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!
by , 4 years ago
| Attachment: | ecm_info_on_off.patch added |
|---|
comment:4 by , 4 years ago
then please the replaced Version with clean -->module-webif.c
Best regards! & @all thanks for patch
comment:6 by , 4 years ago
| Owner: | set to |
|---|---|
| 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 by , 4 years ago
| Resolution: | → fixed |
|---|---|
| Status: | assigned → closed |
Implemented in revision 11539.
What's the disables good for?
Most SKIN's use this info.