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

Error during checkout of Closure project #630

Open
jundaz opened this issue Nov 30, 2024 · 4 comments
Open

Error during checkout of Closure project #630

jundaz opened this issue Nov 30, 2024 · 4 comments

Comments

@jundaz
Copy link

jundaz commented Nov 30, 2024

Hello,
I am using Defects4j 3.0.1, java 11
Errors that looks like the following appear during checkout of any bugs (both buggy and fixed) in Closure project:
sed: can't read /path/to/checkout/directory/Closure_68/fixed/lib/rhino/build.properties: No such file or directory
sed: can't read /path/to/checkout/directory/Closure_68/fixed/lib/rhino/build.properties: No such file or directory
sed: can't read /path/to/checkout/directory/Closure_68/fixed/lib/rhino/src/mozilla/js/rhino/build.properties: No such file or directory
sed: can't read /path/to/checkout/directory/Closure_68/fixed/lib/rhino/src/mozilla/js/rhino/build.properties: No such file or directory
some only reports the latter 2 during checkout(i.e. only with Mozilla in path)
It doesn't seem to affect compiling but I do want to confirm is this behavior expected or is there any way to resolve this.

Thank you very much.

@rjust
Copy link
Owner

rjust commented Nov 30, 2024

Hi @jundaz,

Thanks for reporting this. You can ignore these errors. (These build.properties files need to be patched, if they exist. They don't exist for this and other Closure versions.) We'll add a check to avoid these confusing errors.

Thanks,
René

@markro49
Copy link
Collaborator

markro49 commented Dec 3, 2024

This is fixed in upcoming version 4. Would you like me to back port the fix?

@jundaz
Copy link
Author

jundaz commented Dec 3, 2024

This is fixed in upcoming version 4. Would you like me to back port the fix?

That would be nice

@markro49
Copy link
Collaborator

markro49 commented Jan 21, 2025

This was fixed with pull request #634 that has been merged.

@markro49 markro49 assigned rjust and unassigned markro49 Jan 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants