Claudenw commented on PR #2842: URL: https://github.com/apache/cassandra/pull/2842#issuecomment-1874990481
@michaelsembwever short answer is: no, I did not use the script. I have fixed the issues. As I noted in Slack (and here for public record) I didn't know that what it was for. I will have to update the 'change dependencies' documentation I recently submitted. In addition when I tried to run the script on a `Linux 6.2.0-39-generic #40~22.04.1-Ubuntu SMP PREEMPT_DYNAMIC Thu Nov 16 10:53:04 UTC 2 x86_64 x86_64 x86_64 GNU/Linux` system it failed with 2 errors `sed: can't read s/cassandra\.classpath\.jars\">.*<\/property>/cassandra\.classpath\.jars\">NEW_CLASSPATH<\/property>/: No such file or directory` and a much longer one from the next line with the CLASSPATH but same `No such file or directory` error. The SED comands in the script have blank files for script names (the single quotes after the -i as noted below: ``` sed -i '' 's/cassandra\.classpath\.jars\">.*<\/property>/cassandra\.classpath\.jars\">NEW_CLASSPATH<\/property>/' $DIR/project.xml sed -i '' "s@NEW_CLASSPATH@"$CLASSPATH"@" $DIR/project.xml ``` If I remove the single quotes it works on my system but I am reticent to include this in this fix. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
