Re: [GNC-dev] Fwd: gnucash master: Multiple changes pushed

Store the register sort and filter > settings in .gcm metadata file (requires at least GnuCash 3.3) > > > -- Forwarded message - > From: Robert Fewell > Date: Thu, 14 May 2020 at 09:36 > Subject: gnucash master: Multiple changes pushed > To: >

[GNC-dev] Fwd: gnucash master: Multiple changes pushed

) -- Forwarded message - From: Robert Fewell Date: Thu, 14 May 2020 at 09:36 Subject: gnucash master: Multiple changes pushed To: Updated via https://github.com/Gnucash/gnucash/commit/6fb50d22 (commit) via https://github.com/Gnucash/gnucash/commit/bf9c4441 (commit) via

Re: [GNC-dev] gnucash master: Multiple changes pushed

Op woensdag 13 mei 2020 14:50:45 CEST schreef Christopher Lam: > Hi the branch name was originally 'maint-'. Github doesn't allow changing > branch names without creating a new PR. So, next time I'll just need to > amend the merge commit message (and also add a #link to PR). That would also be

Re: [GNC-dev] gnucash master: Multiple changes pushed

Hi the branch name was originally 'maint-'. Github doesn't allow changing branch names without creating a new PR. So, next time I'll just need to amend the merge commit message (and also add a #link to PR). On Wed, 13 May 2020 at 12:21, Geert Janssens wrote: > Op woensdag 13 mei 2020 14:17:20

Re: [GNC-dev] gnucash master: Multiple changes pushed

Op woensdag 13 mei 2020 14:17:20 CEST schreef Geert Janssens: > Op dinsdag 12 mei 2020 16:06:05 CEST schreef Christopher Lam: > > Updated via https://github.com/Gnucash/gnucash/commit/57fe0515 (commit) > > > > via https://github.com/Gnucash/gnucash/commit/b4d7386d (commit) > > >

Re: [GNC-dev] gnucash master: Multiple changes pushed

Op dinsdag 12 mei 2020 16:06:05 CEST schreef Christopher Lam: > Updatedvia https://github.com/Gnucash/gnucash/commit/57fe0515 > (commit) >via https://github.com/Gnucash/gnucash/commit/b4d7386d (commit) > from https://github.com/Gnucash/gnucash/commit/ebd9db89 (commit) >

Re: [GNC-dev] gnucash master: Multiple changes pushed

Op vrijdag 1 mei 2020 13:03:53 CEST schreef Robert Fewell: > commit 3602753f5093313592dee6acf92b821fc604c976 > Author: Robert Fewell <14ubo...@gmail.com> > Date: Fri May 1 11:54:58 2020 +0100 > > Remove some Gtk version checks in code > > With the requirement for Gtk+ 3.22, the version

Re: [GNC-dev] gnucash master: Multiple changes pushed

Op maandag 7 oktober 2019 04:51:57 CEST schreef John Ralls: > > On Oct 6, 2019, at 1:02 PM, Geert Janssens > > wrote: > > > > This push didn't make it to github due to a network glitch at > > code.gnucash.org (unable to resolve github.com). > > > > Whoever does the next push to master, please

Re: [GNC-dev] gnucash master: Multiple changes pushed

> On Oct 6, 2019, at 1:02 PM, Geert Janssens wrote: > > This push didn't make it to github due to a network glitch at > code.gnucash.org > (unable to resolve github.com). > > Whoever does the next push to master, please take care these commits are in > your local master branch before

Re: [GNC-dev] gnucash master: Multiple changes pushed

And that turned out to be me (with a commit to maint). Problem solved. Geert Op zondag 6 oktober 2019 22:02:21 CEST schreef Geert Janssens: > This push didn't make it to github due to a network glitch at > code.gnucash.org (unable to resolve github.com). > > Whoever does the next push to

Re: [GNC-dev] gnucash master: Multiple changes pushed

This push didn't make it to github due to a network glitch at code.gnucash.org (unable to resolve github.com). Whoever does the next push to master, please take care these commits are in your local master branch before doing so. Otherwise you might be tempted to force push to fix the

Re: gnucash master: Multiple changes pushed

> On Aug 19, 2017, at 6:05 PM, Geert Janssens > wrote: > > On zaterdag 19 augustus 2017 17:33:03 CEST John Ralls wrote: >> commit 6a359f86a07b2adcf6233cafc7531656e02cc6df >> Author: John Ralls >> Date: Wed Aug 16 23:16:42 2017 +0200 >> >>

Re: gnucash master: Multiple changes pushed

On zaterdag 19 augustus 2017 17:33:03 CEST John Ralls wrote: > commit 6a359f86a07b2adcf6233cafc7531656e02cc6df > Author: John Ralls > Date: Wed Aug 16 23:16:42 2017 +0200 > > Fix mac looking in wrong directory for libgncmod-backend-dbi.dylib. > > diff --git

Re: gnucash master: Multiple changes pushed

> On Feb 21, 2017, at 9:16 AM, David T. wrote: > > >> On Feb 21, 2017, at 10:06 PM, John Ralls wrote: >> >> >>> On Feb 21, 2017, at 8:43 AM, Derek Atkins wrote: >>> >>> It's weird that this isn't caught by the *compiler* -- isn't

Re: gnucash master: Multiple changes pushed

> On Feb 21, 2017, at 10:06 PM, John Ralls wrote: > > >> On Feb 21, 2017, at 8:43 AM, Derek Atkins wrote: >> >> It's weird that this isn't caught by the *compiler* -- isn't an unused >> parameter a warning/error? > > It is by default, but we turn it off

Re: gnucash master: Multiple changes pushed

> On Feb 21, 2017, at 8:43 AM, Derek Atkins wrote: > > Geert Janssens > writes: > >> Hi John, >> >> Op maandag 20 februari 2017 19:06:19 CET schreef John Ralls: >> >>> @@ -246,7 +246,7 @@ inline GncRational

Re: gnucash master: Multiple changes pushed

Op dinsdag 21 februari 2017 11:43:35 CET schreef Derek Atkins: > Geert Janssens writes: > > Hi John, > > > > Op maandag 20 februari 2017 19:06:19 CET schreef John Ralls: > >> @@ -246,7 +246,7 @@ inline GncRational operator+(GncRational a, GncInt128 > >> b) } > >> >

Re: gnucash master: Multiple changes pushed

Geert Janssens writes: > Hi John, > > Op maandag 20 februari 2017 19:06:19 CET schreef John Ralls: > >> @@ -246,7 +246,7 @@ inline GncRational operator+(GncRational a, GncInt128 b) >> } >> inline GncRational operator+(GncInt128 a, GncRational b) >> { >> -return

Re: gnucash master: Multiple changes pushed

Op dinsdag 21 februari 2017 07:07:58 CET schreef John Ralls: > Interesting. I didn't get that particular failure on Travis; I got it on my > local Ubuntu 14.04 VM which I use to debug Travis failures. It didn't even > occur to me that the exception (which isn't the one the standard requires > and

Re: gnucash master: Multiple changes pushed

Hi John, Op maandag 20 februari 2017 19:06:19 CET schreef John Ralls: > @@ -246,7 +246,7 @@ inline GncRational operator+(GncRational a, GncInt128 b) > } > inline GncRational operator+(GncInt128 a, GncRational b) > { > -return b + GncRational(a, 1); > +return GncRational(a, 1) + a; > }

Re: gnucash master: Multiple changes pushed & Windows

Hi John Sent from Mail for Windows 10 > On Feb 5, 2017, at 12:04 PM, Stephen Brown > wrote: > > Hi All, > > I am still unable to get a clean build of gnucash under Windows 10. > > Does it work for others or is it

Re: gnucash master: Multiple changes pushed & Windows

> On Feb 5, 2017, at 12:04 PM, Stephen Brown > wrote: > > Hi All, > > I am still unable to get a clean build of gnucash under Windows 10. > > Does it work for others or is it still my installation? I just did a clean build starting from `cscript.exe

gnucash master: Multiple changes pushed & Windows

Hi All, I am still unable to get a clean build of gnucash under Windows 10. Does it work for others or is it still my installation? Cheers Stephen Brown ___ gnucash-devel mailing list gnucash-devel@gnucash.org

Re: gnucash master: Multiple changes pushed

On Thu, Feb 2, 2017 at 11:09 PM, Geert Janssens wrote: > Op donderdag 2 februari 2017 16:35:02 CET schreef Alex Aycinena: > > Geert, > > > > I build on Fedora 25 and after this commit configure fails with: > > > > checking for main in -lgtest_main... no > > checking

Re: gnucash master: Multiple changes pushed

Op donderdag 2 februari 2017 16:35:02 CET schreef Alex Aycinena: > Geert, > > I build on Fedora 25 and after this commit configure fails with: > > checking for main in -lgtest_main... no > checking for /usr/src/gtest/src/gtest-all.cc... no > checking for /usr/src/gtest/gtest-main.cc... no >

Re: gnucash master: Multiple changes pushed

> On Feb 2, 2017, at 4:35 PM, Alex Aycinena wrote: > > On Thu, Feb 2, 2017 at 10:03 AM, Geert Janssens > wrote: > >> Updated via https://github.com/Gnucash/gnucash/commit/659c96eb (commit) >> via

Re: gnucash master: Multiple changes pushed

On Thu, Feb 2, 2017 at 10:03 AM, Geert Janssens wrote: > Updated via https://github.com/Gnucash/gnucash/commit/659c96eb (commit) > via https://github.com/Gnucash/gnucash/commit/4fdacb9b (commit) > via

Re: gnucash master: Multiple changes pushed

> On Oct 9, 2016, at 6:48 PM, Geert Janssens wrote: > > On Sunday 09 October 2016 04:54:11 John Ralls wrote: >> Updated via https://github.com/Gnucash/gnucash/commit/20c0271e >> (commit) via https://github.com/Gnucash/gnucash/commit/b00694fd >> (commit) from

Re: gnucash master: Multiple changes pushed

On Sunday 09 October 2016 04:54:11 John Ralls wrote: > Updatedvia https://github.com/Gnucash/gnucash/commit/20c0271e > (commit) via https://github.com/Gnucash/gnucash/commit/b00694fd > (commit) from https://github.com/Gnucash/gnucash/commit/6e132d83 > (commit) > > > > commit

Re: gnucash master: Multiple changes pushed

On Sun, Jun 12, 2016 at 11:48 AM, John Ralls wrote: > > > > > > > I'm not using guid's in my code. These errors are pulled in because of > the Account.h header > > that gets included somewhere. > > > > I saw one commit on the guid_header branch is including guid.hpp >

Re: gnucash master: Multiple changes pushed

> On Jun 12, 2016, at 8:20 AM, Geert Janssens > wrote: > > On Thursday 09 June 2016 18:42:28 John Ralls wrote: >> Updated via https://github.com/Gnucash/gnucash/commit/2bbc1a19 >> (commit) via https://github.com/Gnucash/gnucash/commit/949ff816 >> (commit)

Re: gnucash master: Multiple changes pushed

On Thursday 09 June 2016 18:42:28 John Ralls wrote: > Updatedvia https://github.com/Gnucash/gnucash/commit/2bbc1a19 > (commit) via https://github.com/Gnucash/gnucash/commit/949ff816 > (commit) via https://github.com/Gnucash/gnucash/commit/ce190e34 > (commit) via

Re: gnucash master: Multiple changes pushed

Thanks for the heads-up, Alex. Fixed in master now. Regards, Geert On Monday 02 November 2015 13:29:50 Alex Aycinena wrote: > Geert, > > I get this make failure on this change: > > /home/gnucash-dev/gitcheckouts/gnucash-clean/src/gnome/gnc-plugin-page > -register.c: In function

Re: gnucash master: Multiple changes pushed

Geert, I get this make failure on this change: /home/gnucash-dev/gitcheckouts/gnucash-clean/src/gnome/gnc-plugin-page-register.c: In function 'gnc_plugin_page_register_get_query': /home/gnucash-dev/gitcheckouts/gnucash-clean/src/gnome/gnc-plugin-page-register.c:1817:443: error: 'return' with no

Re: gnucash master: Multiple changes pushed

On Aug 10, 2015, at 3:05 AM, John Ralls jra...@ceridwen.us wrote: I pushed the change Alex suggested to maint and that got my other change synced to Github so everything should be OK now. I pushed changes to maint and master that I think fix things up. They replace my change to

Re: gnucash master: Multiple changes pushed

On Aug 9, 2015, at 11:51 PM, Mike Alexander m...@umich.edu wrote: --On August 9, 2015 at 10:36:52 AM +0100 John Ralls jra...@ceridwen.us wrote: m On Aug 8, 2015, at 5:00 PM, gnucash-patches-requ...@gnucash.org wrote: commit 1e16ba6c14ccec6222a8fe798e7824285fd49cac Author: Mike

Re: gnucash master: Multiple changes pushed

On Aug 10, 2015, at 7:30 AM, John Ralls jra...@ceridwen.us wrote: On Aug 9, 2015, at 11:51 PM, Mike Alexander m...@umich.edu wrote: --On August 9, 2015 at 10:36:52 AM +0100 John Ralls jra...@ceridwen.us wrote: m On Aug 8, 2015, at 5:00 PM, gnucash-patches-requ...@gnucash.org

Re: gnucash master: Multiple changes pushed

m On Aug 8, 2015, at 5:00 PM, gnucash-patches-requ...@gnucash.org wrote: commit 1e16ba6c14ccec6222a8fe798e7824285fd49cac Author: Mike Alexander m...@umich.edu Date: Fri Aug 7 19:43:12 2015 -0400 Quote the date returned by get_quote_time so Scheme can read it. commit

Re: gnucash master: Multiple changes pushed

--On August 9, 2015 at 10:36:52 AM +0100 John Ralls jra...@ceridwen.us wrote: m On Aug 8, 2015, at 5:00 PM, gnucash-patches-requ...@gnucash.org wrote: commit 1e16ba6c14ccec6222a8fe798e7824285fd49cac Author: Mike Alexander m...@umich.edu Date: Fri Aug 7 19:43:12 2015 -0400 Quote the date

Re: gnucash master: Multiple changes pushed

Geert Janssens geert.gnuc...@kobaltwit.be writes: On Tuesday 04 August 2015 12:00:06 Derek Atkins wrote: Geert, This change seems like the wrong direction; the point of including the book in the API was to reduce the need for global variables/storage. I'd like to see GnuCash move more

Re: gnucash master: Multiple changes pushed

On Aug 5, 2015, at 6:05 PM, Derek Atkins warl...@mit.edu wrote: Geert Janssens geert.gnuc...@kobaltwit.be writes: On Tuesday 04 August 2015 12:00:06 Derek Atkins wrote: Geert, This change seems like the wrong direction; the point of including the book in the API was to reduce the need

Re: gnucash master: Multiple changes pushed

On Tuesday 04 August 2015 12:00:06 Derek Atkins wrote: Geert, This change seems like the wrong direction; the point of including the book in the API was to reduce the need for global variables/storage. I'd like to see GnuCash move more towards being able to have multiple files open

Re: gnucash master: Multiple changes pushed

Geert, This change seems like the wrong direction; the point of including the book in the API was to reduce the need for global variables/storage. I'd like to see GnuCash move more towards being able to have multiple files open simultaneously, not further away from it. I clearly missed the

Re: gnucash master: Multiple changes pushed

John, How do you explain this push ? It's a push to master yet with several commits from maint. I don't see a merge happen from maint to master though ? Did you cherry-pick them ? Geert On Monday 08 December 2014 00:55:58 John Ralls wrote: Updatedvia

Re: gnucash master: Multiple changes pushed

On Dec 8, 2014, at 11:33 AM, Geert Janssens geert.gnuc...@kobaltwit.be wrote: John, How do you explain this push ? It's a push to master yet with several commits from maint. I don't see a merge happen from maint to master though ? Did you cherry-pick them ? Geert, No, I merged,

Re: gnucash master: Multiple changes pushed

On Friday 22 August 2014 19:45:07 John Ralls wrote: Updatedvia https://github.com/Gnucash/gnucash/commit/dd10aab5 (commit) via https://github.com/Gnucash/gnucash/commit/3f8718ea (commit) via https://github.com/Gnucash/gnucash/commit/bad092a4 (commit) from

Re: gnucash master: Multiple changes pushed

Jralls, Thanks, these will be very helpful! I'm sure I wouldn't have come up with that perl replacement for grep. In Christ, Aaron Laws On Fri, Aug 8, 2014 at 6:15 PM, John Ralls jra...@code.gnucash.org wrote: Updated via https://github.com/Gnucash/gnucash/commit/e25c2a34 (commit)

Re: gnucash master: Multiple changes pushed

On Jul 9, 2014, at 3:05 AM, Tim Janssen tjans...@gmail.com wrote: Hello gnucash-devel, I've been following this thread and trying to compile GnuCash myself and have been getting the same error about unrecognised command line option but have gotten different results through experimenting.

Re: gnucash master: Multiple changes pushed

On Mon, Jul 7, 2014 at 11:38 AM, John Ralls jra...@ceridwen.us wrote: On Jul 7, 2014, at 4:03 PM, Sumit Bhardwaj bhardw...@gmail.com wrote: Hi John, Sumit, I don't understand how you concluded that the AM_CXXFLAGS are being included when make builds gfec.c, nor do I understand why you

Re: gnucash master: Multiple changes pushed

On Jul 8, 2014, at 4:43 PM, Sumit Bhardwaj bhardw...@gmail.com wrote: On Mon, Jul 7, 2014 at 11:38 AM, John Ralls jra...@ceridwen.us wrote: On Jul 7, 2014, at 4:03 PM, Sumit Bhardwaj bhardw...@gmail.com wrote: Hi John, Sumit, I don't understand how you concluded that the AM_CXXFLAGS

Re: gnucash master: Multiple changes pushed

On Jul 7, 2014, at 2:07 AM, Sumit Bhardwaj bhardw...@gmail.com wrote: Just a quick update so everyone is on the same page: problem is not with macro (autogen.sh) or configure. It seems the call for gfec.c mixes CFLAGS and CXXFLAGS and then cc1plus or cc1 throws error. Diagnosis below. I

Re: gnucash master: Multiple changes pushed

On Sunday 06 July 2014 19:29:11 John Ralls wrote: On Jul 6, 2014, at 7:15 PM, Geert Janssens janssens-ge...@telenet.be wrote: John, I will be away on holidays soon. If you think my commit should be reverted, feel free to do so. I doubt I will have much online time available while I'm

Re: gnucash master: Multiple changes pushed

On Jul 7, 2014, at 4:16 PM, Geert Janssens janssens-ge...@telenet.be wrote: On Sunday 06 July 2014 19:29:11 John Ralls wrote: On Jul 6, 2014, at 7:15 PM, Geert Janssens janssens-ge...@telenet.be wrote: John, I will be away on holidays soon. If you think my commit should be

Re: gnucash master: Multiple changes pushed

On Jul 7, 2014, at 4:03 PM, Sumit Bhardwaj bhardw...@gmail.com wrote: Hi John, Sumit, I don't understand how you concluded that the AM_CXXFLAGS are being included when make builds gfec.c, nor do I understand why you think that it's using g++. The command make emitted when it failed

Re: gnucash master: Multiple changes pushed

On Jul 5, 2014, at 11:34 PM, Sumit Bhardwaj bhardw...@gmail.com wrote: Sure, John. Here you go. Interestingly, I didn't find conftest.c in the directory tree. 'tree | grep conftest.c' gave no output. Thanks, Sumit configure:25408: checking whether C++ compiler accepts

Re: gnucash master: Multiple changes pushed

Hi John, That works - to an extent. gcc fails and the current configuration is: -Werror -Wdeclaration-after-statement -Wno-pointer-sign -D_FORTIFY_SOURCE=2 Configure doesn't fail, but that the flags are far fewer than earlier. Is this, what we want to go ahead with? If anyone wants to look at

Re: gnucash master: Multiple changes pushed

On Jul 6, 2014, at 6:38 PM, Sumit Bhardwaj bhardw...@gmail.com wrote: Hi John, That works - to an extent. gcc fails and the current configuration is: -Werror -Wdeclaration-after-statement -Wno-pointer-sign -D_FORTIFY_SOURCE=2 Configure doesn't fail, but that the flags are far fewer than

Re: gnucash master: Multiple changes pushed

John, I will be away on holidays soon. If you think my commit should be reverted, feel free to do so. I doubt I will have much online time available while I'm gone. Geert On Sunday 06 July 2014 10:05:27 John Ralls wrote: On Jul 5, 2014, at 11:34 PM, Sumit Bhardwaj bhardw...@gmail.com wrote:

Re: gnucash master: Multiple changes pushed

On Jul 6, 2014, at 7:15 PM, Geert Janssens janssens-ge...@telenet.be wrote: John, I will be away on holidays soon. If you think my commit should be reverted, feel free to do so. I doubt I will have much online time available while I'm gone. Geert, Yeah, you're not likely to have good

Re: gnucash master: Multiple changes pushed

Just a quick note on my problems with compiling master today with Wno-non-literal-null-conversion. Compile kept on failing with the error message gcc error: unrecognized command line option referring to the Wno-non-literal-null-conversion flag. My gcc version is 4.8.3 20140624 (Red Hat 4.8.3-1)

Re: gnucash master: Multiple changes pushed

On July 5, 2014 at 2:43 AM Sumit Bhardwaj bhardw...@gmail.com wrote: Just a quick note on my problems with compiling master today with Wno-non-literal-null-conversion. Compile kept on failing with the error message gcc error: unrecognized command line option referring to the

Re: gnucash master: Multiple changes pushed

Hi John, It fails in first C compilation. I do have to do a make disclean to get the error. Rebuilding with just uncommenting in configure doesn't throw error which is why I didn't see it earlier. I have pasted the error stack below. Do let me know if you want me to do any more experiments. For

Re: gnucash master: Multiple changes pushed

On July 5, 2014 at 11:35 AM Sumit Bhardwaj bhardw...@gmail.com wrote: Hi John, It fails in first C compilation. I do have to do a make disclean to get the error. Rebuilding with just uncommenting in configure doesn't throw error which is why I didn't see it earlier. I have pasted the

Re: gnucash master: Multiple changes pushed

Sure, John. Here you go. Interestingly, I didn't find conftest.c in the directory tree. 'tree | grep conftest.c' gave no output. Thanks, Sumit configure:25408: checking whether C++ compiler accepts -Wno-deprecated-register configure:25427: g++ -c -g -O2 -std=gnu++11 -Werror

Re: gnucash master: Multiple changes pushed

On July 3, 2014 at 5:22 PM Geert Janssens janssens-ge...@telenet.be wrote: On Thursday 03 July 2014 20:07:46 John Ralls wrote: On Jul 3, 2014, at 7:25 PM, Geert Janssens janssens-ge...@telenet.be wrote: Same issue but this time it may need some more careful testing.

Re: gnucash master: Multiple changes pushed

On Friday 04 July 2014 03:03:27 jra...@ceridwen.us wrote: On July 3, 2014 at 5:22 PM Geert Janssens janssens-ge...@telenet.be wrote: On Thursday 03 July 2014 20:07:46 John Ralls wrote: On Jul 3, 2014, at 7:25 PM, Geert Janssens janssens-ge...@telenet.be wrote: Same issue

Re: gnucash master: Multiple changes pushed

On Wednesday 02 July 2014 17:01:10 John Ralls wrote: Updatedvia https://github.com/Gnucash/gnucash/commit/29923b1f (commit) via https://github.com/Gnucash/gnucash/commit/773326b7 (commit) from https://github.com/Gnucash/gnucash/commit/d3389828 (commit) commit

Re: gnucash master: Multiple changes pushed

Hmm, my previous mail got truncated by the list server for some reason. Anyway the errors continue for a while with yet more undefined references to std::xyz functions. This was with a clean autogen/configure/make sequence in an empty build directory. If I do the same with gcc it works fine.

Re: gnucash master: Multiple changes pushed

On Jul 3, 2014, at 3:58 PM, Geert Janssens janssens-ge...@telenet.be wrote: Hmm, my previous mail got truncated by the list server for some reason. Anyway the errors continue for a while with yet more undefined references to std::xyz functions. This was with a clean

Re: gnucash master: Multiple changes pushed

On Thursday 03 July 2014 16:09:28 John Ralls wrote: On Jul 3, 2014, at 3:58 PM, Geert Janssens janssens-ge...@telenet.be wrote: Hmm, my previous mail got truncated by the list server for some reason. Anyway the errors continue for a while with yet more undefined references to std::xyz

Re: gnucash master: Multiple changes pushed

On Jul 3, 2014, at 4:16 PM, Geert Janssens janssens-ge...@telenet.be wrote: On Thursday 03 July 2014 16:09:28 John Ralls wrote: On Jul 3, 2014, at 3:58 PM, Geert Janssens janssens-ge...@telenet.be wrote: Hmm, my previous mail got truncated by the list server for some reason. Anyway

Re: gnucash master: Multiple changes pushed

On Thursday 03 July 2014 17:51:42 John Ralls wrote: On Jul 3, 2014, at 4:16 PM, Geert Janssens janssens-ge...@telenet.be wrote: On Thursday 03 July 2014 16:09:28 John Ralls wrote: On Jul 3, 2014, at 3:58 PM, Geert Janssens janssens-ge...@telenet.be wrote: Hmm, my previous mail got

Re: gnucash master: Multiple changes pushed

On Thursday 03 July 2014 18:04:25 Geert Janssens wrote: On Thursday 03 July 2014 17:51:42 John Ralls wrote: On Jul 3, 2014, at 4:16 PM, Geert Janssens janssens-ge...@telenet.be wrote: On Thursday 03 July 2014 16:09:28 John Ralls wrote: On Jul 3, 2014, at 3:58 PM, Geert Janssens

Re: gnucash master: Multiple changes pushed

On Jul 3, 2014, at 6:04 PM, Geert Janssens janssens-ge...@telenet.be wrote: On Thursday 03 July 2014 17:51:42 John Ralls wrote: On Jul 3, 2014, at 4:16 PM, Geert Janssens janssens-ge...@telenet.be wrote: On Thursday 03 July 2014 16:09:28 John Ralls wrote: On Jul 3, 2014, at 3:58 PM,

Re: gnucash master: Multiple changes pushed

On Jul 3, 2014, at 7:25 PM, Geert Janssens janssens-ge...@telenet.be wrote: I have fixed these as they were fairly trivial. clang next chokes in gnc-tree-view-split-reg.c: /kobaltnet/janssege/Development/EclipseGnuCash/GnuCash-git/src/gnome-utils/gnc-tree-view-split-reg.c:5309:33:

Re: gnucash master: Multiple changes pushed

On Thursday 03 July 2014 20:07:46 John Ralls wrote: On Jul 3, 2014, at 7:25 PM, Geert Janssens janssens-ge...@telenet.be wrote: Same issue but this time it may need some more careful testing. TXN_TYPE_NONE may be used in other locations where '\0' is assumed instead of NULL (guile comes to

Re: gnucash master: Multiple changes pushed

--On July 2, 2014 at 5:01:10 PM -0400 John Ralls jra...@code.gnucash.org wrote: Updated via https://github.com/Gnucash/gnucash/commit/29923b1f (commit) via https://github.com/Gnucash/gnucash/commit/773326b7 (commit)from https://github.com/Gnucash/gnucash/commit/d3389828

Re: switching to C++ compiling by -xc++ (was: gnucash master: Multiple changes pushed)

I just updated guid.cpp to use boost, and I was very pleased at the lack of explosions :-) I issued a pull request to jralls, but for general perusal, you can see it here: https://github.com/limitedAtonement/gnucash/commit/3dc5159cb57f25dc606416263bee8b23da3e6ffe In Christ, Aaron Laws On Mon,

Re: gnucash master: Multiple changes pushed

Any reason why the email script didn't print out a 'diff' for this changeset? -derek Geert Janssens gjanss...@code.gnucash.org writes: Updatedvia https://github.com/Gnucash/gnucash/commit/8784b944 (commit) via https://github.com/Gnucash/gnucash/commit/c4bbd145 (commit)

Re: gnucash master: Multiple changes pushed

I noticed this as well with John's c++ related commits. It appears that the mail script will only print diffs for commits that are pushed directly to a branch and not for commits that get merged in. I seem to remember this was explained in the beginning of these mails before John reduced the

Re: switching to C++ compiling by -xc++ (was: gnucash master: Multiple changes pushed)

On Apr 27, 2014, at 7:58 PM, Mike Alexander m...@umich.edu wrote: On Apr 27, 2014, at 5:15 PM, John Ralls jra...@ceridwen.us wrote: However, I don’t think changing files one by one is wise: It will result in libraries with objects compiled with different compilers, and I worry that that

Re: switching to C++ compiling by -xc++ (was: gnucash master: Multiple changes pushed)

--On April 28, 2014 7:47:07 AM -0700 John Ralls jra...@ceridwen.us wrote: I'll make gnucash-bin.cpp after that. Why not do it first? It needs to be done sooner or later and that might avoid some of the explosions. Mike ___

Re: switching to C++ compiling by -xc++ (was: gnucash master: Multiple changes pushed)

On Apr 28, 2014, at 8:51 AM, Mike Alexander m...@umich.edu wrote: --On April 28, 2014 7:47:07 AM -0700 John Ralls jra...@ceridwen.us wrote: I'll make gnucash-bin.cpp after that. Why not do it first? It needs to be done sooner or later and that might avoid some of the explosions. I

Re: switching to C++ compiling by -xc++ (was: gnucash master: Multiple changes pushed)

--On April 28, 2014 9:04:06 AM -0700 John Ralls jra...@ceridwen.us wrote: On Apr 28, 2014, at 8:51 AM, Mike Alexander m...@umich.edu wrote: --On April 28, 2014 7:47:07 AM -0700 John Ralls jra...@ceridwen.us wrote: I'll make gnucash-bin.cpp after that. Why not do it first? It needs to

Re: switching to C++ compiling by -xc++ (was: gnucash master: Multiple changes pushed)

On Apr 28, 2014, at 10:55 AM, Mike Alexander m...@umich.edu wrote: --On April 28, 2014 9:04:06 AM -0700 John Ralls jra...@ceridwen.us wrote: On Apr 28, 2014, at 8:51 AM, Mike Alexander m...@umich.edu wrote: --On April 28, 2014 7:47:07 AM -0700 John Ralls jra...@ceridwen.us wrote:

Re: switching to C++ compiling by -xc++ (was: gnucash master: Multiple changes pushed)

Hi John, in bedf00a160669a86942a1b52f47c65b8c45c9552 you said you changed LibQOF to be compiled as C++, which you did by adding -xc++ to the CFLAGS, but leaving the file suffixes as .c. This isn't nice. The convention throughout all projects that I know so far is that if the file ends in .c,

Re: switching to C++ compiling by -xc++ (was: gnucash master: Multiple changes pushed)

--On April 27, 2014 10:06:24 PM +0200 Christian Stimming christ...@cstimming.de wrote: Can we please trigger the switch from C to C++ by renaming the files, one by one? This makes it clear which files have to adhere to C++ rules and which ones don't have to. Thanks! I agree with this.

Re: switching to C++ compiling by -xc++ (was: gnucash master: Multiple changes pushed)

On Apr 27, 2014, at 2:01 PM, Mike Alexander m...@umich.edu wrote: --On April 27, 2014 10:06:24 PM +0200 Christian Stimming christ...@cstimming.de wrote: Can we please trigger the switch from C to C++ by renaming the files, one by one? This makes it clear which files have to adhere to C++

Re: switching to C++ compiling by -xc++ (was: gnucash master: Multiple changes pushed)

On Apr 27, 2014, at 5:15 PM, John Ralls jra...@ceridwen.us wrote: However, I don’t think changing files one by one is wise: It will result in libraries with objects compiled with different compilers, and I worry that that will introduce difficult to understand bugs. That’s not a matter for

Re: gnucash master: Multiple changes pushed

Hi John, in the commit message you wrote gwenhywfar-4.10.0 breaks the build, but have you actually checked 4.10.0? I only have a mingw cross-compiler here which runs fine with 4.10.0 but IIRC also did 4.11.0 correctly, so I'm not sure whether you've observed something that I didn't. Surely

Re: gnucash master: Multiple changes pushed

On Mar 31, 2014, at 1:51 PM, Christian Stimming christ...@cstimming.de wrote: Hi John, in the commit message you wrote gwenhywfar-4.10.0 breaks the build, but have you actually checked 4.10.0? I only have a mingw cross-compiler here which runs fine with 4.10.0 but IIRC also did 4.11.0