Replace pkg_resources

[fschulze]

Just importing pkg_resources becomes slower and slower the more packages are installed. Even on fast machines with SSDs this can add a second or more to the startup time.

[fschulze]

This will most likely fail for packages where the resources have to be extracted, as the used context wrapper will cleanup upon exit and the current *TemplateFile implementations currently try to access the file later. There should most likely be a separate class like ResourceTemplate and the loader should return that instead.

[malthe]

@fschulze seems this is almost there? what remains here before we can merge

[fschulze]

See my last comment. It fails for packages which have the templates zipped. I'm also not sure whether that case is covered by tests yet (and how hard such tests would be to implement/simulate).

[malthe]

@fschulze that makes sense – perhaps we should shift this way of loading a file-based template to the *File classes – instead of having it in the loader?

But one question, doesn't the current code also fail with zipped templates?

[fschulze]

I will most likely work on that this week.

[coveralls]

Pull Request Test Coverage Report for Build 6840603550

• 126 of 138 (91.3%) changed or added relevant lines in 7 files are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage increased (+0.2%) to 87.43%

---

Changes Missing Coverage	Covered Lines	Changed/Added Lines	%
src/chameleon/template.py	42	48	87.5%
src/chameleon/tests/test_loader.py	75	81	92.59%

Totals
Change from base Build 6839785589:	0.2%
Covered Lines:	4283
Relevant Lines:	4792

---

💛 - Coveralls

[fschulze]

The spec is now stored on the template and handled properly. The filename attribute is now marked as deprecated.