Pull Requests

A place to discuss and plan OpenSource xTalk (not exclusively LCC based)
and Community Builds of LCC ...Ask NOT what xTalk can do for you...
Get involved you DO have something to contribute, no matter your skillset!

Forum rules
A place to discuss and plan OpenSource xTalk (not exclusively LCC based) and Community Builds of LCC
Ask NOT what xTalk can do for you... get involved you DO have something to contribute, no matter your skillset!
mwieder
Posts: 130
Joined: Sun Jun 04, 2023 3:32 am
Location: Berkeley, CA, US, Earth
Contact:

Re: Pull Requests

Post by mwieder »

I just mangled some things and pushed the repo again.
If you pull the main repo from github you should be good to go.
mwieder
Posts: 130
Joined: Sun Jun 04, 2023 3:32 am
Location: Berkeley, CA, US, Earth
Contact:

Re: Pull Requests

Post by mwieder »

The thing about "not having to deal with git" is that git/github etc is the way to submit changes, especially if there's more than one person working on a system of files.

We can be making changes simultaneously and then merging the changes is mostly a matter of accepting pull requests, no more having to go line-by-line through the code figuring out what's changed.

Stick with it. You'll thank me later.
User avatar
tperry2x
Posts: 2788
Joined: Tue Dec 21, 2021 9:10 pm
Location: Somewhere in deepest darkest Norfolk, England
Contact:

Re: Pull Requests

Post by tperry2x »

Anyway, the point of me wanting to edit bits is to try out a compile, to see if I can fix the MacOS systemversion function, and also any other annoying bugs that are embedded in-engine.
I think this would fix it (untested of course), but needless to say - the mac recompile would also be a separate issue altogether.
I'll pin this here for a possible MacOS systemversion fix. Can you add it to your git code as a possible fix?
mwieder wrote: Tue Jun 25, 2024 5:42 pm git/github etc is the way to submit changes, especially if there's more than one person working on a system of files.
If you love git or understand it, that is.
mwieder wrote: Tue Jun 25, 2024 5:42 pm We can be making changes simultaneously and then merging the changes is mostly a matter of accepting pull requests
I don't understand 'pull requests'. WTAF, but it's like someone has placed a brick wall in front of me. I've tried and tried to get my head around it, but I'm sorry - I just don't get it.

It feels like a solution for a non-existent problem.
I like the old-fashioned, old-skool way:
"Here's the source. Download it. Make a note of what you changed as you change it. Tell me the line numbers and the file path.... test, verify... done. Here's my changed file..."
mwieder wrote: Tue Jun 25, 2024 5:42 pm no more having to go line-by-line through the code figuring out what's changed.
That's what I prefer, to edit the code directly on the computer - as I know what lines i changed.
mwieder wrote: Tue Jun 25, 2024 5:42 pm Stick with it. You'll thank me later.
Hahahaha, I really won't :lol:
systemversion mac edit.7z
(38.59 KiB) Downloaded 124 times
User avatar
tperry2x
Posts: 2788
Joined: Tue Dec 21, 2021 9:10 pm
Location: Somewhere in deepest darkest Norfolk, England
Contact:

Re: Pull Requests

Post by tperry2x »

Don't worry about it, just look at dskmac.cpp in that archive, and all my changes have the word tperry next to them.
User avatar
tperry2x
Posts: 2788
Joined: Tue Dec 21, 2021 9:10 pm
Location: Somewhere in deepest darkest Norfolk, England
Contact:

Re: Pull Requests

Post by tperry2x »

One last go, and compiling is still very broken:
next-error.png
next-error.png (39.13 KiB) Viewed 5196 times
Now seems I'm missing "grammar_full.c" which then spawns other errors. :|

Also worth mentioning probably. While it was trying (and failing) to compile - I grabbed this screenshot.
Chromium is used for the CEF browser engine, but Chromium 74 was released early 2019.
how-old.png
how-old.png (36.1 KiB) Viewed 5177 times
mwieder
Posts: 130
Joined: Sun Jun 04, 2023 3:32 am
Location: Berkeley, CA, US, Earth
Contact:

Re: Pull Requests

Post by mwieder »

Yeah. I haven't been able to upgrade the CEF library past version 74. Since it doesn't work on linux anyway I'm not too concerned. :roll: Someday we'll figure out a good replacement library. I see LC is moving away from it on OSX and Windows anyway.
mwieder
Posts: 130
Joined: Sun Jun 04, 2023 3:32 am
Location: Berkeley, CA, US, Earth
Contact:

Re: Pull Requests

Post by mwieder »

Meanwhile, to make life with git easier, do these two things. I can no longer imagine git work without them:

1. install git-aware-terminal through npm. This will give you a nice color display of the current branch and status. (I had to upgrade my older node and npm first).
https://www.npmjs.com/package/git-aware-terminal

2. install git-completion-bash from https://github.com/git/git/blob/master/ ... etion.bash
just that one file. Mine is saved in my home directory as .git-completion-bash.
Then add the following to your .bashrc file

Code: Select all

if [ -f ~/.git-completion.bash ]; then
    . ~/.git-completion.bash
fi
This will give you shortcut aliases for git commands and type-ahead for completion.

Execute this command in a terminal window:

Code: Select all

source ~/.bash_profile
Then open a new terminal prompt and navigate to the livecode directory.
User avatar
tperry2x
Posts: 2788
Joined: Tue Dec 21, 2021 9:10 pm
Location: Somewhere in deepest darkest Norfolk, England
Contact:

Re: Pull Requests

Post by tperry2x »

Thank you for the file link you sent me. As promised, I downloaded it and tried it out this evening.
I'm running into the following errors. (hard to grab as they scroll past so fast, and there's only so much scrollback available).
a.png
a.png (12.58 KiB) Viewed 5157 times
b.png
b.png (13.09 KiB) Viewed 5157 times
c.png
c.png (30.69 KiB) Viewed 5157 times
d.png
d.png (51.77 KiB) Viewed 5157 times
mwieder
Posts: 130
Joined: Sun Jun 04, 2023 3:32 am
Location: Berkeley, CA, US, Earth
Contact:

Re: Pull Requests

Post by mwieder »

Interesting that you're seeing errors from cc1plus. That's the backend compiler to g++.
What version of g++ do you have installed? I'm using 10.5.0.
User avatar
tperry2x
Posts: 2788
Joined: Tue Dec 21, 2021 9:10 pm
Location: Somewhere in deepest darkest Norfolk, England
Contact:

Re: Pull Requests

Post by tperry2x »

Not sure at the moment, (whatever comes with xUbuntu 24 at the time of writing this). I'll try and check tomorrow evening. Many thanks
Post Reply

Who is online

Users browsing this forum: No registered users and 1 guest