| Summary: | Add vbox as a recognized xml mimetype | ||
|---|---|---|---|
| Product: | TDE | Reporter: | Darrell <darrella> |
| Component: | tdelibs | Assignee: | Timothy Pearson <kb9vqf> |
| Status: | RESOLVED FIXED | ||
| Severity: | normal | CC: | bugwatch, darrella, michele.calgaro |
| Priority: | P5 | ||
| Version: | R14.0.0 [Trinity] | ||
| Hardware: | Other | ||
| OS: | Linux | ||
| Compiler Version: | TDE Version String: | ||
| Application Version: | Application Name: | ||
| Bug Depends on: | |||
| Bug Blocks: | 1779 | ||
| Attachments: |
Patch to add *.vbox to tdelibs/kate/data/xml.xml
Patch to add *.vbox to tdelibs/mimetypes/text/xml.desktop |
||
Created attachment 1813 [details]
Patch to add *.vbox to tdelibs/mimetypes/text/xml.desktop
I tested the patch, works well here. .vbox files are now open with syntax highlight automatically enabled. IMO, you can push it to GIT. Which patch did you test? Which patch is preferred? I posted both patches because I don't know which is preferred. (In reply to comment #3) > Which patch did you test? Which patch is preferred? I posted both patches > because I don't know which is preferred. Sorry if it wasn't clear Darrell, yesterday I was up way to late. I actually tested both patches. The first patch suffices in Kate. The second patch alone does not enable syntax highlighting in Kate. IMO the first patch *only* should be pushed ti GIT. I don't think the second patch should be pushed because vbox files could have different file associations compared to xml files. Look for example at rss files: their extension is included in xml.xml but they have their own mimetype rss.desktop file. Eventually you could create a vbox.desktop entry in the mimetypes folder if you needed it. Patch from attachment 1812 [details] pushed to git in commit c5189a79.
|
Created attachment 1812 [details] Patch to add *.vbox to tdelibs/kate/data/xml.xml For years the VirtualBox developers assigned xml as the file extension to all VirtualBox configuration files, which are indeed xml files. I don't know which VirtualBox version, I think 4.x, they changed that file extension from xml to vbox. The declaration at top of the vbox files is clear that the file is an xml file. Unfortunately neither the kate xml.xml syntax file recognizes the vbox extension nor does the Trinity file association scheme recognize vbox as an xml mimetype. Attached are two proposed patches. One modifies the kate xml.xml file. The second modifies the Trinity file association scheme to add vbox to the list of known xml mimetype file extensions.