I made a change to fix a bug with the default plugin type when appending a new row to the table which caused the conflict. This where I noticed the printf() debugging statement. I also noticed that it appears that you removed a scroll to row call and I'm not sure how to reconcile the conflict. Did you remove the scroll to row call on purpose or wasn't it necessary. I don't want to break that behavior.
On 11/20/2017 04:29 PM, Oliver Walters wrote: > Wayne, > > I think it is a whitespace issue, does it apply if you add > --ignore-whitespace to git-am ? > > On Tue, Nov 21, 2017 at 1:03 AM, Wayne Stambaugh <stambau...@gmail.com > <mailto:stambau...@gmail.com>> wrote: > > Oliver, > > No go yet again. It looks like my commit 8b2b1381 is causing a conflict > with patch 2 so please rebase your patches. I also noticed a printf() > debugging statement in patch 2. Please remove this and make sure there > are not any other printf() statements in your patches. Sorry about > asking you to fix these issues but I really don't have the time to fix > patches right now. > > Cheers, > > Wayne > > On 11/20/2017 1:20 AM, Oliver Walters wrote: > > Wayne, > > > > I'm at a loss too. git am fails on 0002 but git apply works fine on each > > individual patch. > > > > On Mon, Nov 20, 2017 at 3:29 AM, Wayne Stambaugh <stambau...@gmail.com > <mailto:stambau...@gmail.com> > > <mailto:stambau...@gmail.com <mailto:stambau...@gmail.com>>> wrote: > > > > Oliver, > > > > Still no luck. Did you possible do something to mess up the > commit > > ordering? I am applying these patches on top a clean master > branch so > > I'm not sure what is going on here. I'm getting the following > error > > from `git am` when I attempt to merge patch 2: > > > > Applying: Toggle LIB_TABLE_ROW enabled/disabled in grid editor > > error: patch failed: eeschema/dialogs/dialog_sym_lib_table.cpp:350 > > error: eeschema/dialogs/dialog_sym_lib_table.cpp: patch does > not apply > > error: patch failed: include/lib_table_grid.h:32 > > error: include/lib_table_grid.h: patch does not apply > > Patch failed at 0001 Toggle LIB_TABLE_ROW enabled/disabled in grid > > editor > > The copy of the patch that failed is found in: > .git/rebase-apply/patch > > When you have resolved this problem, run "git am --continue". > > If you prefer to skip this patch, run "git am --skip" instead. > > To restore the original branch and stop patching, run "git am > --abort". > > > > If any one has any ideas, I'm open to suggestion. If these > patches were > > generated properly, there should be no issues applying them. > > > > Cheers, > > > > Wayne > > > > On 11/18/2017 07:11 PM, Oliver Walters wrote: > > > Wayne > > > > > > Please find updated patch set attached. I have rebased and > built from > > > commit b6884d and it all works fine. > > > > > > Thanks > > > > > > On Sun, Nov 19, 2017 at 4:42 AM, Wayne Stambaugh > <stambau...@gmail.com <mailto:stambau...@gmail.com> > <mailto:stambau...@gmail.com <mailto:stambau...@gmail.com>> > > > <mailto:stambau...@gmail.com <mailto:stambau...@gmail.com> > <mailto:stambau...@gmail.com <mailto:stambau...@gmail.com>>>> wrote: > > > > > > Oliver, > > > > > > I just tried to apply your patches and ran into some > issues. > > Patch 1 > > > applies but patch 2 fails. Would you please rebase your > > patches so I > > > can get them merged as soon as possible. > > > > > > Thanks, > > > > > > Wayne > > > > > > On 11/15/2017 06:41 AM, Oliver Walters wrote: > > > > Wayne, et al, > > > > > > > > I am really liking the way that the new symbol table > works! > > Thanks for > > > > the huge effort that has gone into this. > > > > > > > > One thing that I have noticed is that when opening (for > > e.g.) the > > > > component chooser, the UI hangs while all the libraries > > load. Previously > > > > there was a progress dialog which at least informed > the user > > what was > > > > going on. > > > > > > > > I have re-implemented this dialog in the attached > patch set. > > > > > > > > Further, I have also implemented a way to individually > > enable / disable > > > > each row in the library tables (this works for SYMBOL_LIB > > and FP_LIB). > > > > > > > > This is based on the idea by Tomasz. Example > screenshot below: > > > > > > > > Inline image 1 > > > > > > > > I have tweaked the base LIB_TABLE_GRID code such that > > disabled entries > > > > are greyed out and made italic. The enabled / disabled > status is > > > > persistent in the sym/fp_lib_table files. Older version of > > xxx_lib_table > > > > files are read with all rows enabled by default. > > > > > > > > Please find patch set attached. > > > > > > > > Regards, > > > > Oliver > > > > > > > > > > > > > > > > _______________________________________________ > > > > Mailing list: https://launchpad.net/~kicad-developers > <https://launchpad.net/~kicad-developers> > > <https://launchpad.net/~kicad-developers > <https://launchpad.net/~kicad-developers>> > > > <https://launchpad.net/~kicad-developers > <https://launchpad.net/~kicad-developers> > > <https://launchpad.net/~kicad-developers > <https://launchpad.net/~kicad-developers>>> > > > > Post to : kicad-developers@lists.launchpad.net > <mailto:kicad-developers@lists.launchpad.net> > > <mailto:kicad-developers@lists.launchpad.net > <mailto:kicad-developers@lists.launchpad.net>> > > > <mailto:kicad-developers@lists.launchpad.net > <mailto:kicad-developers@lists.launchpad.net> > > <mailto:kicad-developers@lists.launchpad.net > <mailto:kicad-developers@lists.launchpad.net>>> > > > > Unsubscribe : https://launchpad.net/~kicad-developers > <https://launchpad.net/~kicad-developers> > > <https://launchpad.net/~kicad-developers > <https://launchpad.net/~kicad-developers>> > > > <https://launchpad.net/~kicad-developers > <https://launchpad.net/~kicad-developers> > > <https://launchpad.net/~kicad-developers > <https://launchpad.net/~kicad-developers>>> > > > > More help : https://help.launchpad.net/ListHelp > <https://help.launchpad.net/ListHelp> > > <https://help.launchpad.net/ListHelp > <https://help.launchpad.net/ListHelp>> > > > <https://help.launchpad.net/ListHelp > <https://help.launchpad.net/ListHelp> > > <https://help.launchpad.net/ListHelp > <https://help.launchpad.net/ListHelp>>> > > > > > > > > > > _______________________________________________ > > > Mailing list: https://launchpad.net/~kicad-developers > <https://launchpad.net/~kicad-developers> > > <https://launchpad.net/~kicad-developers > <https://launchpad.net/~kicad-developers>> > > > <https://launchpad.net/~kicad-developers > <https://launchpad.net/~kicad-developers> > > <https://launchpad.net/~kicad-developers > <https://launchpad.net/~kicad-developers>>> > > > Post to : kicad-developers@lists.launchpad.net > <mailto:kicad-developers@lists.launchpad.net> > > <mailto:kicad-developers@lists.launchpad.net > <mailto:kicad-developers@lists.launchpad.net>> > > > <mailto:kicad-developers@lists.launchpad.net > <mailto:kicad-developers@lists.launchpad.net> > > <mailto:kicad-developers@lists.launchpad.net > <mailto:kicad-developers@lists.launchpad.net>>> > > > Unsubscribe : https://launchpad.net/~kicad-developers > <https://launchpad.net/~kicad-developers> > > <https://launchpad.net/~kicad-developers > <https://launchpad.net/~kicad-developers>> > > > <https://launchpad.net/~kicad-developers > <https://launchpad.net/~kicad-developers> > > <https://launchpad.net/~kicad-developers > <https://launchpad.net/~kicad-developers>>> > > > More help : https://help.launchpad.net/ListHelp > <https://help.launchpad.net/ListHelp> > > <https://help.launchpad.net/ListHelp > <https://help.launchpad.net/ListHelp>> > > > <https://help.launchpad.net/ListHelp > <https://help.launchpad.net/ListHelp> > > <https://help.launchpad.net/ListHelp > <https://help.launchpad.net/ListHelp>>> > > > > > > > > > > > > _______________________________________________ Mailing list: https://launchpad.net/~kicad-developers Post to : kicad-developers@lists.launchpad.net Unsubscribe : https://launchpad.net/~kicad-developers More help : https://help.launchpad.net/ListHelp