By default, Bugzilla does not search the list of RESOLVED bugs.
You can force it to do so by putting the upper-case word ALL in front of your search query, e.g.: ALL tdelibs
We recommend searching for bugs this way, as you may discover that your bug has already been resolved and fixed in a later release.
Bug 2302 - Add file types to kate ini.xml syntax highlighting
Summary: Add file types to kate ini.xml syntax highlighting
Status: RESOLVED FIXED
Alias: None
Product: TDE
Classification: Unclassified
Component: tdelibs (show other bugs)
Version: R14.0.0 [Trinity]
Hardware: Other Linux
: P5 normal
Assignee: Michele Calgaro
URL:
Depends on:
Blocks: R14.0.1
  Show dependency tree / graph
 
Reported: 2015-01-05 17:26 CST by Darrell
Modified: 2015-08-27 10:50 CDT (History)
4 users (show)

See Also:
Compiler Version:
TDE Version String:
Application Version:
Application Name:

Attachments
Patch for updating ini.xml filetypes (734 bytes, patch)
2015-01-05 17:26 CST, Darrell 		Details | Diff
Patch for updating ini.xml filetypes (730 bytes, patch)
2015-05-31 12:21 CDT, Darrell 		Details | Diff
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Darrell 2015-01-05 17:26:21 CST 
Created attachment 2412 [details]
Patch for updating ini.xml filetypes

File types added:

*rc: basically any Trinity rc config file

*.service;*.target: systemd config files, using the same classic INI file format
Comment 1 Michele Calgaro 2015-01-08 22:09:22 CST 
I think *rc is too much generic for considering a file as ini file. There may be perfectly valid files whose name ends in rc and be something different.
Comment 2 Darrell 2015-05-31 12:21:25 CDT 
Created attachment 2513 [details]
Patch for updating ini.xml filetypes

Updated patch includes only systemd *.target and *.service file types.
Comment 3 Michele Calgaro 2015-06-01 08:24:02 CDT 
Patch looks good. IMO we could add this for R14.0.1. Slavek, what's your opinion?
Comment 4 Michele Calgaro 2015-06-09 18:13:25 CDT 
> Patch looks good. IMO we could add this for R14.0.1. Slavek, what's your opinion?

I rephrase: it should be added to R14.1.x since technically it is an improvement and not a bug fix (although small enough, granted)
Comment 5 Michele Calgaro 2015-06-09 18:40:49 CDT 
After discussion with Slavek, we decided that the patch can be pushed to both the main trunk and the R14.0.x branch.

Darrell, can you push the patch to both branches or do you prefer we do ourself (in such case please provide a "sign-off" clause as specified by the TDE CLA)?
Comment 6 Darrell 2015-06-10 07:35:20 CDT 
Best one of you push the patch. I am out of the loop and do not have a local 14.x.x branch.
Comment 7 Michele Calgaro 2015-06-10 18:43:39 CDT 
Pushed in commit 8b1fec8 (master) and 4317437 (r14.0.x).