Skip to content
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

Revert "Revert pgtkconn removal" #11

Merged
merged 1 commit into from Jan 20, 2020
Merged

Revert "Revert pgtkconn removal" #11

merged 1 commit into from Jan 20, 2020

Conversation

masm11
Copy link
Owner

@masm11 masm11 commented Jan 20, 2020

I can't reproduce the freeze,
so I revert #10 as I wrote in #9.

@masm11 masm11 merged commit 2fe3355 into pgtk Jan 20, 2020
@masm11
Copy link
Owner Author

masm11 commented Jan 20, 2020

and freezing...

what mistake am i doing...?

@masm11
Copy link
Owner Author

masm11 commented Jan 20, 2020

There seemed to be a build mistake.

@fejfighter
Copy link

I had it for a little while late last night on one of my dev branches,

a make bootstrap & make install seemed to help

must have been linking something in a broken way.

@masm11
Copy link
Owner Author

masm11 commented Jan 21, 2020

@fejfighter
Hi,
thanks for the advice.

I debugged and resolved. I pushed commits to pgtk.

I confirmed:

  • stable cursor blink
  • no freeze
  • sane cpu load even if with < /dev/null

@fejfighter
Copy link

Sorry,

I misread your comment this morning,

I did not realise that you had found a better solution, good work, it's much more elegant.

@masm11
Copy link
Owner Author

masm11 commented Mar 18, 2020

Hi, @fejfighter ,

I want to merge this fork to mainline.

When I do, I'll need to assign copyright of my code in this fork to FSF.

Also, this fork contains your code.
Do you agree that you assign copyright of your code in this fork to FSF?

@fejfighter
Copy link

Hey,

That would be great,
I assume that I will need to sign paperwork as well?

for now: Yes I do agree to assign my code to the FSF.

When are you hoping to merge it in? I guess some time after 27 is released?
There are still a couple of missing features that would be nice to handle.

@masm11
Copy link
Owner Author

masm11 commented Mar 19, 2020

@fejfighter

I assume that I will need to sign paperwork as well?

I don't know about the detail.
When I tell the developer team about the merge request, they will tell me the detail, I think.

for now: Yes I do agree to assign my code to the FSF.

Thank you.

When are you hoping to merge it in? I guess some time after 27 is released?

master is different branch from 27, so anytime may be ok, I think.

There are still a couple of missing features that would be nice to handle.

OK, I'll wait for them.

@skangas
Copy link

skangas commented Apr 27, 2020

I assume that I will need to sign paperwork as well?

Yes, the fastest way is to e-mail emacs-devel@gnu.org and ask for the copyright assignment form to get the process started.

Best regards,
Stefan Kangas

@dgutov
Copy link

dgutov commented Apr 27, 2020

There is one other thing. The patches submitted to Emacs require commit messages written to the GNU standard.

We could turn this whole branch into one combined patch with a title "Pure GTK port added" and a mechanical description of the changes. But if you'd like to keep the changes history, it would require some more work it splitting it up and writing messages for each. At it seems like @fejfighter might be the person for that job, if interested.

@fejfighter
Copy link

@dgutov early work in #23

@dgutov
Copy link

dgutov commented May 12, 2020

@fejfighter Thank you. Am I correct to assume that you haven't worked on the commit messages themselves yet? The Emacs project requires them in a particular format, exemplified here: https://www.gnu.org/software/emacs/manual/html_node/emacs/Format-of-ChangeLog.html#Format-of-ChangeLog

Just in case this is a surprise: we do have commands in Emacs that help in creating such entries, so you don't type every file name, function name, paren and colon by hand.

@fejfighter
Copy link

@dgutov, correct, my focus was on rebasing cleanly onto master and folding in fixups while trying to maintain some "thematic" commits, I may have been a little ambitious and will need to compress more of them.

I think I would have been more surprised if there wasn't a command, but I hand not thought of using it for the commits, so thanks!

@masm11
Copy link
Owner Author

masm11 commented May 12, 2020

@dgutov Thanks for your advice.

I learned how to do that in magit today.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants