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

Ghost changes in commits #106

Closed
machiav3lli opened this issue Feb 26, 2022 · 19 comments · Fixed by #172
Closed

Ghost changes in commits #106

machiav3lli opened this issue Feb 26, 2022 · 19 comments · Fixed by #172
Labels
bug Something isn't working

Comments

@machiav3lli
Copy link

In the last I noticed that sometimes GittyUp add changes out of nothing to the staged commits, I wasn't able even to see them in the staged block. Here's the three latest examples:

It's also good to mention that I wasn't able to reproduce it intentionally…

@stefanknotzer
Copy link
Collaborator

Assuming 'its fix' is the correction to the staging failure: in every case lines were erroneous deleted? So you had to add the deletions to fix the staging problem?

@machiav3lli
Copy link
Author

@stefanknotzer nope, this is what I meant by it being ghost. Nothing happend to the file, but in the commit's diff it showed up, after refreshing it was possible to recommit the missed code.

It's fully random as you can see in the cited commits, sometimes it's a whole file, sometimes it's just couple of lines.

@Murmele
Copy link
Owner

Murmele commented Mar 2, 2022

On which are you working?

@machiav3lli
Copy link
Author

AUR's gittyup-git: https://aur.archlinux.org/packages/gittyup-git

@Murmele
Copy link
Owner

Murmele commented Mar 5, 2022

I think I have seen this too.

@Murmele
Copy link
Owner

Murmele commented Mar 10, 2022

In this scenario, add_subdirectory(Spice) does not show if it is staged or not. When opening a different git tool it shows that this line is staged. Maybe it is related to this bug here.
image

@Murmele
Copy link
Owner

Murmele commented Mar 15, 2022

In this scenario, add_subdirectory(Spice) does not show if it is staged or not. When opening a different git tool it shows that this line is staged. Maybe it is related to this bug here. image

Hmm not reproducable anymore

@machiav3lli
Copy link
Author

Since my last writing, I had it only once, and still not reproducable. Maybe there's something changed partly fixing it or it's just me being more cautious with each staging and commit.

@ghost
Copy link

ghost commented Mar 17, 2022

No in this area nothing changed so the issue is still there

@ghost
Copy link

ghost commented Mar 17, 2022

I think it is a missmatch between between staged and unstaged file passed to the Filewidget.

@exactly-one-kas exactly-one-kas added the bug Something isn't working label Mar 29, 2022
@Murmele
Copy link
Owner

Murmele commented Apr 1, 2022

Seems to be different on windows and on linux. I was able to reproduce it on windows, but on linus the behavior is different

@Murmele
Copy link
Owner

Murmele commented Apr 1, 2022

I have now a reproducable example. Hope i can fix it soon

@machiav3lli
Copy link
Author

Ah, nice!

@Murmele Murmele mentioned this issue Apr 2, 2022
@Murmele
Copy link
Owner

Murmele commented Apr 2, 2022

@machiav3lli
Can you try if you are able to reproduce with the build artifacts from here: #122

@Murmele
Copy link
Owner

Murmele commented Apr 20, 2022

I close it for now. If it happens again, please reopen it

@Murmele Murmele closed this as completed Apr 20, 2022
@machiav3lli
Copy link
Author

Hm.. it worked just fine till yesterday, when I had it again and till now am unable to reproduce it (Nothing changed, I'm still using the latest dev AUR gittyup-git).

@Murmele
Copy link
Owner

Murmele commented May 22, 2022

@machiav3lli I think now I found it :) Can you have a look on the pullrequest and the description there. Maybe you remember if you had a scenario like this.
#172

Please open this issue again if this does not fix you problem

@Murmele Murmele closed this as completed May 22, 2022
@machiav3lli
Copy link
Author

Thanks, I'll do. And please consider refreshing the AUR releases (either update the non-dev gittyup-git or do a formal release)

@Murmele
Copy link
Owner

Murmele commented May 23, 2022

I am planning to create a new release next days. Currently we don't support officially AUR releases.

There is a discussion ongoing: #167

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants