janl commented on PR #6: URL: https://github.com/apache/couchdb-admin/pull/6#issuecomment-2453408516
For example: one very important thing to me is validating checksums, especially the GPG one since that needs manual inspection, but that output is lost in the stdout stream. I also don’t need the whole “do a test build for me” section as I have the macOS binary build scripts for that, but I can see how this can be useful, but also, I don’t feel like writing a cross-platform (unixy at least) build script, or maybe that is exactly what we should be doing, but that’d duplicate a lot of CI work and needs to keep maintained. IF we were to decide to do this, I’d suggest we put that bit into a separate script. All that said, I’m not sure if: - we should merge this as-is as it only helps folks who know what they are doing and if we want folks to test things properly, they better do it by hand - OR we fix this up a little bit so it is useful for some folks - OR make this the bells-and-whistles cross-platform script that we officially recommend folks use? I’m sorry for making a lot of noise about a thing that might in the end not be worth merging after all. wdyt? -- 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]
