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