search

flame-org / Modules

Nette modules on the Steroids
http://flame-org.github.io/Modules/
GNU Lesser General Public License v3.0
13 stars 7 forks source link

Licence #12

Closed jirinapravnik closed 10 years ago

jirinapravnik commented 10 years ago

Nechceš to uvolnit pod nějakou mírnější licencí, aby se to dalo použít i do komerčních projektů? Mit licence např? Nebo alespoň LGPL?

jsifalda commented 10 years ago

Pull a není problém :)

jirinapravnik commented 10 years ago

Pull bych klidně udělal, ale to asi bude lepší, když si zvolíš ty, pod čím to chceš uvolnit:-) Btw možná to jde přepnout přímo v GitHubu, aspoň při zakládání repa to nabízí navolit si licenci a pak jí to nakopíruje...

jsifalda commented 10 years ago

Upraveno na LGPL :)

jirinapravnik commented 10 years ago

super, díky! Jinak jsem koukal, že jste Flame\Modules hodně uhladili a zeštíhlili. Hodně dobrá práce. Jen jste mě trochu naštvali odebráním ConfigProvideru, ale to byla trošku prasárničk no:)

jsifalda commented 10 years ago

Děkujeme! :) Jako ConfigProvider to měl sečtené, už od začátku :P (Omlouvám se za to.) btw, co si potřeboval tím configem nastavovat, že ti nestačejí implementované providery? Možná by se s tím dalo něco udělat...

jirinapravnik commented 10 years ago

JJ, vím, že už v prvním blog postu jsi od něj varoval:-)

No já to třeba používal pro nastavení nějakých parametrů, services specifikcé pro moduly (to se dá řešit celkem v pohodě přímo v compilerextension), ale taky třeba pro webloader. ostatně viz zde: http://forum.nette.org/cs/17798-compilerextension-zpracovani-neon#p125151

jsifalda commented 10 years ago

A co tohle https://github.com/flame-org/Modules/commit/91db75b91ead7bca4563e7273a6cf5b2e516c3cd, nepomůže to alespoň trošku? :)

Edit: Samozřejmě, parametry by se měli nastavovat přes $defaults v compileru, ale IParametersProvider by měl sloužit pro definování "konstant" pro daný modul (extension)

Edit2: Má to docela sílu - dovoluje to např. přepsat parametry typu appDir, productionMode atd., což se může hodit při skládání app.

jirinapravnik commented 10 years ago

U těch parametrů já to potřebuji spíše tak, že mám nějaké defaults a pak to překryju v tom hlavním app.config nicméně tohle by podle mě ani nemělo Flame\Modules řešit, to mám nějak pořešeno sám.

Spíš mě trápí to s tím webloaderem apod. Ale jak jsem se dočetl, zatí mto nemá řešení, tak zkusím něco vymyslet sám:-)

jsifalda commented 10 years ago

Dobře :)