I'd like to suggest that the .gitignore recently pushed to the public repo be removed, and that Koha not enforce any conventions via .gitignore . -- Ryan Higgins LibLime * Open-Source Solutions for Libraries Featuring KohaZOOM ILS 888-564-2457 x704 _______________________________________________ Koha-devel mailing list Koha-devel@nongnu.org http://lists.nongnu.org/mailman/listinfo/koha-devel
"Ryan Higgins" <ryan.higgins@liblime.com> wrote:
I'd like to suggest that the .gitignore recently pushed to the public repo be removed, and that Koha not enforce any conventions via .gitignore .
I think this is a decision for QA (is it still Chris covering that? I found it mentioned on-list last October but not on the website) but Ryan should be able to remove it once in a local branch without much problem. Best wishes, -- MJ Ray (slef) Webmaster for hire, statistician and online shop builder for a small worker cooperative http://www.ttllp.co.uk/ http://mjr.towers.org.uk/ (Notice http://mjr.towers.org.uk/email.html) tel:+44-844-4437-237 _______________________________________________ Koha-devel mailing list Koha-devel@nongnu.org http://lists.nongnu.org/mailman/listinfo/koha-devel
On Mon, Apr 21, 2008 at 8:23 PM, MJ Ray <mjr@phonecoop.coop> wrote:
"Ryan Higgins" <ryan.higgins@liblime.com> wrote:
I'd like to suggest that the .gitignore recently pushed to the public repo be removed, and that Koha not enforce any conventions via .gitignore .
I think this is a decision for QA (is it still Chris covering that? I found it mentioned on-list last October but not on the website) but Ryan should be able to remove it once in a local branch without much problem.
Nope not me anymore, when I left Liblime, I could no longer get the mail sent to patches@koha.org I think its been decided by others that someone else is doing QA. The patches now go to a mailing list, but I'm not sure who is doing QA Chris _______________________________________________ Koha-devel mailing list Koha-devel@nongnu.org http://lists.nongnu.org/mailman/listinfo/koha-devel
MJ Ray a écrit :
"Ryan Higgins" <ryan.higgins@liblime.com> wrote:
I'd like to suggest that the .gitignore recently pushed to the public repo be removed, and that Koha not enforce any conventions via .gitignore .
I think this is a decision for QA (is it still Chris covering that? I found it mentioned on-list last October but not on the website) but Ryan should be able to remove it once in a local branch without much problem.
Best wishes,
Hi, I took the QA position, even though had little time to do thorough tests of all the patches that are sent daily... And there has been quite a lot of important ones those days. About .gitignore, I really donot see what would be wrong this little commit. It doesnot impact development and would allow people not to bother about 'faked' diffs when committing things. I agree it is not a "customer" useful commit. Adding it to git would just have you commit yours on your local branch and fetch/rebase instead of pulling. But is this not what we all do ? Question may be : does it HAVE to be there ? No. Question may be : should it be there ? Maybe, for people who donot know git enough to be aware of .gitignore Since it is nicer not to have the list of all the patches or modified files when you commit things. You say Koha should not enforce conventions. I had rather have clear conventions stated so that all the people which comes could know the way to contribute, coding and naming conventions, like any other big project rather than none. For instance, we MUST use ODLIS words for templates. The problem of conventions has sometimes been debated on this list with no definite position. But I still think enforcing conventions is important, all the more than we are providing 2 MARC standards and may be willing to open Koha to other standards. For instance, it could prevent us to use in our code $record->title() for a MARC record, since this is not MARC-flavour agnostic and would allow us to work along and satisfy our customers. -- Henri-Damien LAURENT _______________________________________________ Koha-devel mailing list Koha-devel@nongnu.org http://lists.nongnu.org/mailman/listinfo/koha-devel
On Fri, Apr 18, 2008 at 1:43 PM, Ryan Higgins <ryan.higgins@liblime.com> wrote:
I'd like to suggest that the .gitignore recently pushed to the public repo be removed, and that Koha not enforce any conventions via .gitignore .
Hi Ryan - We chatted a bit about this on IRC, but I'm not sure if I understood that we ever came to any conclusions. Some of the points I recall that we discussed are: * It seems that you're not supposed to put personal things like *~ to skip emacs backup files in .gitignore. They're supposed to go in .git/info/exclude. It sounds like we're breaking that convention, and I don't think there would be an resistance against fixing this. see <http://www.kernel.org/pub/software/scm/git/docs/gitignore.html> * Temporary files that get made by the build process like .patch or t/run may be OK for .gitignore. I'm not sure if I understood if leaving these things there would be OK for you or not. I think the suggestion here was "stuff that 'make clean' would remove" I don't recall that there was no real strong argument to keep the file in git if it's actually causing problems. If we keep the two points above in mind, do you think that this file will still cause the types of problems that you saw? -Andy _______________________________________________ Koha-devel mailing list Koha-devel@nongnu.org http://lists.nongnu.org/mailman/listinfo/koha-devel
Hi, On Mon, Apr 21, 2008 at 8:04 AM, Andrew Moore <andrew.moore@liblime.com> wrote:
On Fri, Apr 18, 2008 at 1:43 PM, Ryan Higgins <ryan.higgins@liblime.com> wrote: * Temporary files that get made by the build process like .patch or t/run may be OK for .gitignore. I'm not sure if I understood if leaving these things there would be OK for you or not. I think the suggestion here was "stuff that 'make clean' would remove"
A project .gitignore that ignored make and git detritus might look something like this. I'm using patterns tied to the project root so that a Makefile in a project subdirectory is not ignored by default. *.patch /t/run /t/test-config.txt /blib /Makefile /Makefile.old /pm_to_blib Regards, Galen -- Galen Charlton Koha Application Developer LibLime galen.charlton@liblime.com p: 1-888-564-2457 x709 _______________________________________________ Koha-devel mailing list Koha-devel@nongnu.org http://lists.nongnu.org/mailman/listinfo/koha-devel
participants (6)
-
Andrew Moore -
Chris Cormack -
Galen Charlton -
Henri-Damien LAURENT -
MJ Ray -
Ryan Higgins