Skip to content

Several fixes#1

Open
olegbbtr wants to merge 3 commits intosenya:masterfrom
olegbbtr:fixes
Open

Several fixes#1
olegbbtr wants to merge 3 commits intosenya:masterfrom
olegbbtr:fixes

Conversation

@olegbbtr
Copy link
Copy Markdown

@olegbbtr olegbbtr commented Apr 9, 2023

commit 7a072f1 (HEAD -> fixes, fork/fixes)
Author: Oleg Vasilev me@svin.in
Date: Sun Apr 9 16:48:30 2023 +0600

Ignore unexisting commits from meta file

This can happen when there was a rebase after git-check-rebase is done.

Signed-off-by: Oleg Vasilev <me@svin.in>

commit c5cad18
Author: Oleg Vasilev me@svin.in
Date: Sun Apr 9 16:22:35 2023 +0600

Omit absent commits during for hide_checked condition

Suppose we have a "base" range. Many commits are not present there, so this will
count as "not checked", but the truth is that those a simply not upstreamed.

Signed-off-by: Oleg Vasilev <me@svin.in>

commit 1ec7f97
Author: Oleg Vasilev oleg.vasilev@virtuozzo.com
Date: Tue Feb 28 12:11:24 2023 +0600

Fix pair unpacking during meta file parsing

If we have an extra ":" in drop or feature comment, like:
  %drop: no need: already done
The program will crash. prop.split(':', 1) will make at most 1 split.

Signed-off-by: Oleg Vasilev <oleg.vasilev@virtuozzo.com>

olegbbtr and others added 3 commits April 9, 2023 16:50
If we have an extra ":" in drop or feature comment, like:
  %drop: no need: already done
The program will crash. prop.split(':', 1) will make at most 1 split.

Signed-off-by: Oleg Vasilev <oleg.vasilev@virtuozzo.com>
Suppose we have a "base" range. Many commits are not present there, so this will
count as "not checked", but the truth is that those a simply not upstreamed.

Signed-off-by: Oleg Vasilev <me@svin.in>
This can happen when there was a rebase after git-check-rebase is done.

Signed-off-by: Oleg Vasilev <me@svin.in>
@olegbbtr
Copy link
Copy Markdown
Author

@senya thoughts?

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.

1 participant