search

tinted-theming / base16-emacs

Base16 themes for Emacs
MIT License
380 stars 76 forks source link

Avoid conflict with the "screenshot" package #39

Closed tarsius closed 7 years ago

tarsius commented 7 years ago

Because base16-theme's "screenshot.el" is isn't a library intended to be loaded using (require 'screenshot), it also should not provide a feature.

There exists an actual package "screenshot", consisting of a library that provides the feature screenshot. To reduce the risk of conflicting with that, also add the file "screenshots/.nosearch", which makes it much less likely that "screenshot.el" ends up on the load-path.

belak commented 7 years ago

Hm, I guess I'll have to take your word for this. I thought it wouldn't be included in the actual built package because :defaults only included *.el... but this still seems reasonable.

Thanks!

tarsius commented 7 years ago

You are right about that. This change is for the benefit of non-Melpa users and people contributing to this theme package.