-
Notifications
You must be signed in to change notification settings - Fork 6
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
version checked into MASTER has a version of 4.111.3. #605
Comments
Hi Tom -
Could you test 4.112.1 <https://tr4w.net/4.112/tr4w_setup_4_112.1.exe> to
ensure it picked up #605 ??
tks, h
…On Sun, Jun 5, 2022 at 6:43 PM Tom Schaefer ***@***.***> wrote:
I am not certain but it looks like the version in MASTER is 4.111.3. I
believe the current version is 4.112? Was this not committed to MASTER? I
guess I am still not certain how this is built as in my view the steps
would be...
Commit everything to MASTER
Fetch MASTER
Build
I ask as when I went to edit5 the code to remove the STATE tag for a grid,
it was not there so there is something out of step between what was built
and what is committed into MASTER.
—
Reply to this email directly, view it on GitHub
<#605>, or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ABVLCUHPSM6476JDHDF5EMDVNUUR3ANCNFSM5X5W4QJQ>
.
You are receiving this because you are subscribed to this thread.Message
ID: ***@***.***>
|
The version you built previously and put not he website was showing 4.112 so that was not the issue. My question was that the version.pas committed into MASTER did not show 4.112 so it appeared that you were building things that were not in MASTER. That could lead to other issues where a patch is applied to MASTER via pull request but it does not make it to the machine from which you build. I checked the version.pas committed to MASTER and it looks good now. The other curiosity was that the version you build for June has the bug with the STATE being added to ADIF for a GRID, but when I stepped through the version committed to MASTER, it did NOT do this. THat tells me there is a difference in other code too. That is the real issue at heart. |
I installed 4.112.1 and the version is OK but it still emits <STATE:4>GRID. The version in MASTER does not so I guess I will research any changes in this area. Do you recall any? |
Hi Tom - I think a change I was doing for ne9u got into 4.111.3 and put our
two masters out of sync.
I think 112.0 is now ok, can you pull that in as master and redo 604 ??
…On Mon, Jun 6, 2022 at 10:56 AM Tom Schaefer ***@***.***> wrote:
I installed 4.112.1 and the version is OK but it still emits STATE:4GRID.
The version in MASTER does not so I guess I will research any changes in
this area. Do you recall any?
—
Reply to this email directly, view it on GitHub
<#605 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/ABVLCUCCI7DCWRHQ4HPMK5LVNYGS5ANCNFSM5X5W4QJQ>
.
You are receiving this because you commented.Message ID:
***@***.***>
|
The odd thing is the code in master does not have the error so I made no changes. But what you compiled this morning and the previous 4.112 did. I am at a loss to explain that.
Tom
… On Jun 6, 2022, at 1:58 PM, Howie Hoyt ***@***.***> wrote:
Hi Tom - I think a change I was doing for ne9u got into 4.111.3 and put our
two masters out of sync.
I think 112.0 is now ok, can you pull that in as master and redo 604 ??
On Mon, Jun 6, 2022 at 10:56 AM Tom Schaefer ***@***.***>
wrote:
> I installed 4.112.1 and the version is OK but it still emits STATE:4GRID.
> The version in MASTER does not so I guess I will research any changes in
> this area. Do you recall any?
>
> —
> Reply to this email directly, view it on GitHub
> <#605 (comment)>, or
> unsubscribe
> <https://github.com/notifications/unsubscribe-auth/ABVLCUCCI7DCWRHQ4HPMK5LVNYGS5ANCNFSM5X5W4QJQ>
> .
> You are receiving this because you commented.Message ID:
> ***@***.***>
>
—
Reply to this email directly, view it on GitHub <#605 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AC6TWSVPBRHZEKOUPPZG6ELVNY34LANCNFSM5X5W4QJQ>.
You are receiving this because you authored the thread.
|
ok. ii will try to rebuild from 4.111.1 forward to 4.112.0 as master.
On Mon, Jun 6, 2022 at 2:01 PM Tom Schaefer ***@***.***>
wrote:
… The odd thing is the code in master does not have the error so I made no
changes. But what you compiled this morning and the previous 4.112 did. I
am at a loss to explain that.
Tom
> On Jun 6, 2022, at 1:58 PM, Howie Hoyt ***@***.***> wrote:
>
>
> Hi Tom - I think a change I was doing for ne9u got into 4.111.3 and put
our
> two masters out of sync.
> I think 112.0 is now ok, can you pull that in as master and redo 604 ??
>
> On Mon, Jun 6, 2022 at 10:56 AM Tom Schaefer ***@***.***>
> wrote:
>
> > I installed 4.112.1 and the version is OK but it still emits
STATE:4GRID.
> > The version in MASTER does not so I guess I will research any changes
in
> > this area. Do you recall any?
> >
> > —
> > Reply to this email directly, view it on GitHub
> > <#605 (comment)>, or
> > unsubscribe
> > <
https://github.com/notifications/unsubscribe-auth/ABVLCUCCI7DCWRHQ4HPMK5LVNYGS5ANCNFSM5X5W4QJQ
>
> > .
> > You are receiving this because you commented.Message ID:
> > ***@***.***>
> >
> —
> Reply to this email directly, view it on GitHub <
#605 (comment)>, or
unsubscribe <
https://github.com/notifications/unsubscribe-auth/AC6TWSVPBRHZEKOUPPZG6ELVNY34LANCNFSM5X5W4QJQ
>.
> You are receiving this because you authored the thread.
>
—
Reply to this email directly, view it on GitHub
<#605 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/ABVLCUD5BZDGSVCREEMUFQ3VNY4F3ANCNFSM5X5W4QJQ>
.
You are receiving this because you commented.Message ID:
***@***.***>
|
I think 4.111.3 had important fixes though
Principal Solutions Architect
Better Software Solutions, Inc.
727-437-2771
… On Jun 6, 2022, at 2:24 PM, Howie Hoyt ***@***.***> wrote:
ok. ii will try to rebuild from 4.111.1 forward to 4.112.0 as master.
On Mon, Jun 6, 2022 at 2:01 PM Tom Schaefer ***@***.***>
wrote:
> The odd thing is the code in master does not have the error so I made no
> changes. But what you compiled this morning and the previous 4.112 did. I
> am at a loss to explain that.
>
> Tom
>
> > On Jun 6, 2022, at 1:58 PM, Howie Hoyt ***@***.***> wrote:
> >
> >
> > Hi Tom - I think a change I was doing for ne9u got into 4.111.3 and put
> our
> > two masters out of sync.
> > I think 112.0 is now ok, can you pull that in as master and redo 604 ??
> >
> > On Mon, Jun 6, 2022 at 10:56 AM Tom Schaefer ***@***.***>
> > wrote:
> >
> > > I installed 4.112.1 and the version is OK but it still emits
> STATE:4GRID.
> > > The version in MASTER does not so I guess I will research any changes
> in
> > > this area. Do you recall any?
> > >
> > > —
> > > Reply to this email directly, view it on GitHub
> > > <#605 (comment)>, or
> > > unsubscribe
> > > <
> https://github.com/notifications/unsubscribe-auth/ABVLCUCCI7DCWRHQ4HPMK5LVNYGS5ANCNFSM5X5W4QJQ
> >
> > > .
> > > You are receiving this because you commented.Message ID:
> > > ***@***.***>
> > >
> > —
> > Reply to this email directly, view it on GitHub <
> #605 (comment)>, or
> unsubscribe <
> https://github.com/notifications/unsubscribe-auth/AC6TWSVPBRHZEKOUPPZG6ELVNY34LANCNFSM5X5W4QJQ
> >.
> > You are receiving this because you authored the thread.
> >
>
> —
> Reply to this email directly, view it on GitHub
> <#605 (comment)>, or
> unsubscribe
> <https://github.com/notifications/unsubscribe-auth/ABVLCUD5BZDGSVCREEMUFQ3VNY4F3ANCNFSM5X5W4QJQ>
> .
> You are receiving this because you commented.Message ID:
> ***@***.***>
>
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.
|
If you just fetch Master making sure you don’t have any files on your build PC not pulled down by the fetch, you should be good.
Principal Solutions Architect
Better Software Solutions, Inc.
727-437-2771
… On Jun 6, 2022, at 2:24 PM, Howie Hoyt ***@***.***> wrote:
ok. ii will try to rebuild from 4.111.1 forward to 4.112.0 as master.
On Mon, Jun 6, 2022 at 2:01 PM Tom Schaefer ***@***.***>
wrote:
> The odd thing is the code in master does not have the error so I made no
> changes. But what you compiled this morning and the previous 4.112 did. I
> am at a loss to explain that.
>
> Tom
>
> > On Jun 6, 2022, at 1:58 PM, Howie Hoyt ***@***.***> wrote:
> >
> >
> > Hi Tom - I think a change I was doing for ne9u got into 4.111.3 and put
> our
> > two masters out of sync.
> > I think 112.0 is now ok, can you pull that in as master and redo 604 ??
> >
> > On Mon, Jun 6, 2022 at 10:56 AM Tom Schaefer ***@***.***>
> > wrote:
> >
> > > I installed 4.112.1 and the version is OK but it still emits
> STATE:4GRID.
> > > The version in MASTER does not so I guess I will research any changes
> in
> > > this area. Do you recall any?
> > >
> > > —
> > > Reply to this email directly, view it on GitHub
> > > <#605 (comment)>, or
> > > unsubscribe
> > > <
> https://github.com/notifications/unsubscribe-auth/ABVLCUCCI7DCWRHQ4HPMK5LVNYGS5ANCNFSM5X5W4QJQ
> >
> > > .
> > > You are receiving this because you commented.Message ID:
> > > ***@***.***>
> > >
> > —
> > Reply to this email directly, view it on GitHub <
> #605 (comment)>, or
> unsubscribe <
> https://github.com/notifications/unsubscribe-auth/AC6TWSVPBRHZEKOUPPZG6ELVNY34LANCNFSM5X5W4QJQ
> >.
> > You are receiving this because you authored the thread.
> >
>
> —
> Reply to this email directly, view it on GitHub
> <#605 (comment)>, or
> unsubscribe
> <https://github.com/notifications/unsubscribe-auth/ABVLCUD5BZDGSVCREEMUFQ3VNY4F3ANCNFSM5X5W4QJQ>
> .
> You are receiving this because you commented.Message ID:
> ***@***.***>
>
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.
|
as long as i can find the individual fixes for 4.111.3
On Mon, Jun 6, 2022 at 2:25 PM Tom Schaefer ***@***.***>
wrote:
… I think 4.111.3 had important fixes though
Principal Solutions Architect
Better Software Solutions, Inc.
727-437-2771
> On Jun 6, 2022, at 2:24 PM, Howie Hoyt ***@***.***> wrote:
>
>
> ok. ii will try to rebuild from 4.111.1 forward to 4.112.0 as master.
>
> On Mon, Jun 6, 2022 at 2:01 PM Tom Schaefer ***@***.***>
> wrote:
>
> > The odd thing is the code in master does not have the error so I made
no
> > changes. But what you compiled this morning and the previous 4.112
did. I
> > am at a loss to explain that.
> >
> > Tom
> >
> > > On Jun 6, 2022, at 1:58 PM, Howie Hoyt ***@***.***> wrote:
> > >
> > >
> > > Hi Tom - I think a change I was doing for ne9u got into 4.111.3 and
put
> > our
> > > two masters out of sync.
> > > I think 112.0 is now ok, can you pull that in as master and redo 604
??
> > >
> > > On Mon, Jun 6, 2022 at 10:56 AM Tom Schaefer ***@***.***>
> > > wrote:
> > >
> > > > I installed 4.112.1 and the version is OK but it still emits
> > STATE:4GRID.
> > > > The version in MASTER does not so I guess I will research any
changes
> > in
> > > > this area. Do you recall any?
> > > >
> > > > —
> > > > Reply to this email directly, view it on GitHub
> > > > <#605 (comment)>,
or
> > > > unsubscribe
> > > > <
> >
https://github.com/notifications/unsubscribe-auth/ABVLCUCCI7DCWRHQ4HPMK5LVNYGS5ANCNFSM5X5W4QJQ
> > >
> > > > .
> > > > You are receiving this because you commented.Message ID:
> > > > ***@***.***>
> > > >
> > > —
> > > Reply to this email directly, view it on GitHub <
> > #605 (comment)>, or
> > unsubscribe <
> >
https://github.com/notifications/unsubscribe-auth/AC6TWSVPBRHZEKOUPPZG6ELVNY34LANCNFSM5X5W4QJQ
> > >.
> > > You are receiving this because you authored the thread.
> > >
> >
> > —
> > Reply to this email directly, view it on GitHub
> > <#605 (comment)>, or
> > unsubscribe
> > <
https://github.com/notifications/unsubscribe-auth/ABVLCUD5BZDGSVCREEMUFQ3VNY4F3ANCNFSM5X5W4QJQ
>
> > .
> > You are receiving this because you commented.Message ID:
> > ***@***.***>
> >
> —
> Reply to this email directly, view it on GitHub, or unsubscribe.
> You are receiving this because you authored the thread.
—
Reply to this email directly, view it on GitHub
<#605 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/ABVLCUG5ENHOHUISHZNMCQLVNY7C7ANCNFSM5X5W4QJQ>
.
You are receiving this because you are subscribed to this thread.Message
ID: ***@***.***>
|
ok done.
what is a good piece of code to verify 604 took ?
git fetch
remote: Enumerating objects: 33, done.
remote: Counting objects: 100% (33/33), done.
remote: Compressing objects: 100% (9/9), done.
remote: Total 33 (delta 24), reused 32 (delta 24), pack-reused 0
Unpacking objects: 100% (33/33), 12.58 KiB | 74.00 KiB/s, done.
From github.com:n4af/TR4W
* [new branch] 4.111.2 -> origin/4.111.2
* [new branch] 4.111.3 -> origin/4.111.3
* [new branch] 4.112.1 -> origin/4.112.1
* [new branch]
604-in-arrl-digital-contest-the-adif-file-exports-state-with-a-grid-square
->
origin/604-in-arrl-digital-contest-the-adif-file-exports-state-with-a-grid-square
* [new branch] Issue599 -> origin/Issue599
fd33bfc..8bd6b59 master -> origin/master
On Mon, Jun 6, 2022 at 2:27 PM Tom Schaefer ***@***.***>
wrote:
… If you just fetch Master making sure you don’t have any files on your
build PC not pulled down by the fetch, you should be good.
Principal Solutions Architect
Better Software Solutions, Inc.
727-437-2771
> On Jun 6, 2022, at 2:24 PM, Howie Hoyt ***@***.***> wrote:
>
>
> ok. ii will try to rebuild from 4.111.1 forward to 4.112.0 as master.
>
> On Mon, Jun 6, 2022 at 2:01 PM Tom Schaefer ***@***.***>
> wrote:
>
> > The odd thing is the code in master does not have the error so I made
no
> > changes. But what you compiled this morning and the previous 4.112
did. I
> > am at a loss to explain that.
> >
> > Tom
> >
> > > On Jun 6, 2022, at 1:58 PM, Howie Hoyt ***@***.***> wrote:
> > >
> > >
> > > Hi Tom - I think a change I was doing for ne9u got into 4.111.3 and
put
> > our
> > > two masters out of sync.
> > > I think 112.0 is now ok, can you pull that in as master and redo 604
??
> > >
> > > On Mon, Jun 6, 2022 at 10:56 AM Tom Schaefer ***@***.***>
> > > wrote:
> > >
> > > > I installed 4.112.1 and the version is OK but it still emits
> > STATE:4GRID.
> > > > The version in MASTER does not so I guess I will research any
changes
> > in
> > > > this area. Do you recall any?
> > > >
> > > > —
> > > > Reply to this email directly, view it on GitHub
> > > > <#605 (comment)>,
or
> > > > unsubscribe
> > > > <
> >
https://github.com/notifications/unsubscribe-auth/ABVLCUCCI7DCWRHQ4HPMK5LVNYGS5ANCNFSM5X5W4QJQ
> > >
> > > > .
> > > > You are receiving this because you commented.Message ID:
> > > > ***@***.***>
> > > >
> > > —
> > > Reply to this email directly, view it on GitHub <
> > #605 (comment)>, or
> > unsubscribe <
> >
https://github.com/notifications/unsubscribe-auth/AC6TWSVPBRHZEKOUPPZG6ELVNY34LANCNFSM5X5W4QJQ
> > >.
> > > You are receiving this because you authored the thread.
> > >
> >
> > —
> > Reply to this email directly, view it on GitHub
> > <#605 (comment)>, or
> > unsubscribe
> > <
https://github.com/notifications/unsubscribe-auth/ABVLCUD5BZDGSVCREEMUFQ3VNY4F3ANCNFSM5X5W4QJQ
>
> > .
> > You are receiving this because you commented.Message ID:
> > ***@***.***>
> >
> —
> Reply to this email directly, view it on GitHub, or unsubscribe.
> You are receiving this because you authored the thread.
—
Reply to this email directly, view it on GitHub
<#605 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/ABVLCUC2AUF2CIZMOKTX7OTVNY7HXANCNFSM5X5W4QJQ>
.
You are receiving this because you are subscribed to this thread.Message
ID: ***@***.***>
|
I’ll have to check. I have a meeting so it will be a bit later. Thanks
Principal Solutions Architect
Better Software Solutions, Inc.
727-437-2771
… On Jun 6, 2022, at 2:31 PM, Howie Hoyt ***@***.***> wrote:
ok done.
what is a good piece of code to verify 604 took ?
git fetch
remote: Enumerating objects: 33, done.
remote: Counting objects: 100% (33/33), done.
remote: Compressing objects: 100% (9/9), done.
remote: Total 33 (delta 24), reused 32 (delta 24), pack-reused 0
Unpacking objects: 100% (33/33), 12.58 KiB | 74.00 KiB/s, done.
From github.com:n4af/TR4W
* [new branch] 4.111.2 -> origin/4.111.2
* [new branch] 4.111.3 -> origin/4.111.3
* [new branch] 4.112.1 -> origin/4.112.1
* [new branch]
604-in-arrl-digital-contest-the-adif-file-exports-state-with-a-grid-square
->
origin/604-in-arrl-digital-contest-the-adif-file-exports-state-with-a-grid-square
* [new branch] Issue599 -> origin/Issue599
fd33bfc..8bd6b59 master -> origin/master
On Mon, Jun 6, 2022 at 2:27 PM Tom Schaefer ***@***.***>
wrote:
> If you just fetch Master making sure you don’t have any files on your
> build PC not pulled down by the fetch, you should be good.
>
> Principal Solutions Architect
> Better Software Solutions, Inc.
> 727-437-2771
>
> > On Jun 6, 2022, at 2:24 PM, Howie Hoyt ***@***.***> wrote:
> >
> >
> > ok. ii will try to rebuild from 4.111.1 forward to 4.112.0 as master.
> >
> > On Mon, Jun 6, 2022 at 2:01 PM Tom Schaefer ***@***.***>
> > wrote:
> >
> > > The odd thing is the code in master does not have the error so I made
> no
> > > changes. But what you compiled this morning and the previous 4.112
> did. I
> > > am at a loss to explain that.
> > >
> > > Tom
> > >
> > > > On Jun 6, 2022, at 1:58 PM, Howie Hoyt ***@***.***> wrote:
> > > >
> > > >
> > > > Hi Tom - I think a change I was doing for ne9u got into 4.111.3 and
> put
> > > our
> > > > two masters out of sync.
> > > > I think 112.0 is now ok, can you pull that in as master and redo 604
> ??
> > > >
> > > > On Mon, Jun 6, 2022 at 10:56 AM Tom Schaefer ***@***.***>
> > > > wrote:
> > > >
> > > > > I installed 4.112.1 and the version is OK but it still emits
> > > STATE:4GRID.
> > > > > The version in MASTER does not so I guess I will research any
> changes
> > > in
> > > > > this area. Do you recall any?
> > > > >
> > > > > —
> > > > > Reply to this email directly, view it on GitHub
> > > > > <#605 (comment)>,
> or
> > > > > unsubscribe
> > > > > <
> > >
> https://github.com/notifications/unsubscribe-auth/ABVLCUCCI7DCWRHQ4HPMK5LVNYGS5ANCNFSM5X5W4QJQ
> > > >
> > > > > .
> > > > > You are receiving this because you commented.Message ID:
> > > > > ***@***.***>
> > > > >
> > > > —
> > > > Reply to this email directly, view it on GitHub <
> > > #605 (comment)>, or
> > > unsubscribe <
> > >
> https://github.com/notifications/unsubscribe-auth/AC6TWSVPBRHZEKOUPPZG6ELVNY34LANCNFSM5X5W4QJQ
> > > >.
> > > > You are receiving this because you authored the thread.
> > > >
> > >
> > > —
> > > Reply to this email directly, view it on GitHub
> > > <#605 (comment)>, or
> > > unsubscribe
> > > <
> https://github.com/notifications/unsubscribe-auth/ABVLCUD5BZDGSVCREEMUFQ3VNY4F3ANCNFSM5X5W4QJQ
> >
> > > .
> > > You are receiving this because you commented.Message ID:
> > > ***@***.***>
> > >
> > —
> > Reply to this email directly, view it on GitHub, or unsubscribe.
> > You are receiving this because you authored the thread.
>
> —
> Reply to this email directly, view it on GitHub
> <#605 (comment)>, or
> unsubscribe
> <https://github.com/notifications/unsubscribe-auth/ABVLCUC2AUF2CIZMOKTX7OTVNY7HXANCNFSM5X5W4QJQ>
> .
> You are receiving this because you are subscribed to this thread.Message
> ID: ***@***.***>
>
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.
|
I am not certain but it looks like the version in MASTER is 4.111.3. I believe the current version is 4.112? Was this not committed to MASTER? I guess I am still not certain how this is built as in my view the steps would be...
Commit everything to MASTER
Fetch MASTER
Build
I ask as when I went to edit5 the code to remove the STATE tag for a grid, it was not there so there is something out of step between what was built and what is committed into MASTER.
The text was updated successfully, but these errors were encountered: