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.

Sign up for GitHub

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

Jump to bottom

Allow updating .result files #36

Merged
merged 1 commit into from
Sep 6, 2022
Merged

Allow updating .result files #36

merged 1 commit into from
Sep 6, 2022

Conversation

ligurio
Copy link
Member

@ligurio ligurio commented Sep 5, 2022

Follows up #23
Fixes #35

@ligurio ligurio requested a review from locker September 5, 2022 16:15
locker
locker requested changes Sep 6, 2022
View reviewed changes
checkpatch.pl Outdated Show resolved Hide resolved
@ligurio ligurio requested a review from locker September 6, 2022 10:41
locker
locker requested changes Sep 6, 2022
View reviewed changes
checkpatch.pl Outdated Show resolved Hide resolved
@locker locker merged commit 418505c into master Sep 6, 2022
@locker locker deleted the ligurio/open_the_gate branch September 6, 2022 14:58
locker added a commit that referenced this pull request Sep 7, 2022
@locker
Fix new result file check
6860141 
The 'new_file' flag is never cleared so if a patch adds a new file and
modifies an existing result file, we will complain. Fix this: detect
file creation by looking at the old file name (/dev/null if the file is
new).

Follow-up #36
Closes #38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@locker locker locker approved these changes

Assignees

@ligurio ligurio

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Error during to modify existing diff tests
2 participants
@ligurio @locker