pyrus / Pyrus_Developer

Developer tools for building PEAR compatible packages
7 stars 7 forks source link

gp vs. gp-simple #2

Open till opened 13 years ago

till commented 13 years ago

I'd like to propose that the current way gp works, is moved to gp-extended, and that we make gp do the following instead:

helgi commented 13 years ago

If we do anything like that then it is a flag on the gp command. No point in polluting the command namespace for something "gp --simple" or "gp --extended" can do.

till commented 13 years ago

That's fine too. I'd just like it to be simple by default.

The majority of people don't need phar, etc.. I also think we should nudge people towards making pyrus packages. And a fallback to PEAR 1.x is optional.

saltybeagle commented 13 years ago

The output of this command has been simplified before, but obviously not enough for your taste. I'd be fine with this change though.

If we do add an extended option, it should also include the custom commands and custom role portions that were originally in the command.

saltybeagle commented 13 years ago

Right now there's a lot of code that is hardcoded and embedded into other PHP files.

How about converting the skeleton files into a full dir or .tgz file that is un-packed then has some minor string replacements made. This might make it easier to maintain.

helgi commented 13 years ago

Templating type of approach? I think that's really the only way going forward given the type of documents we have and will be adding in the next while.

Anything that helps us get away from the 1500+ line classes PEAR suffered from ;-) Also means it is easier to introduce new files and completely different templates (e.g. JSON instead of the plain text or similar non sense)

saltybeagle commented 13 years ago

Yeah, that sounds great. It could even be a separate repo on Github. The skeleton generator just clones the skeleton repo. This would be customizable of course, so people could fork and modify for their local app skeletons...?

saltybeagle commented 13 years ago

Oh, it would have to be available offline for @cweiske of course. :-D

helgi commented 13 years ago

Yeah, people on hand-cranked connections will need to be able to use this

till commented 13 years ago

I think I found the place, so I'm gonna think about writing a test for this command first and then I refactor the hell out of that function.

I'm thinking I just do plain text files and replace over them. No extracting, etc..

till commented 13 years ago

Btw, Christian got DSL now.

Also, people who live in Ireland shouldn't talk about high speed Internet. Cause they have no idea what that's like. ;-)