You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@couchdb.apache.org by GitBox <gi...@apache.org> on 2020/04/07 07:57:12 UTC
[GitHub] [couchdb-pkg] sklassen opened a new pull request #68: A template
for compiling under feature/core20
sklassen opened a new pull request #68: A template for compiling under feature/core20
URL: https://github.com/apache/couchdb-pkg/pull/68
## Overview
For your consideration. This is how I built couchdb for snap using core-20 and supports SM60.
Three big changes.
i) snapcraft now builds directly from the tarball and using the core20 libraries.
ii) couchdb is installed directly into /snap/couchdb/xx/ so no more duplicate subdirectories
iii) the `database_dir` and `view_index_dir` are now explicitly set in the local.ini: This is the most troublesome part of the installation. The layout link is flakey when using underscores and it also removes confusion by making the path absolute in the fauxton client.
And, I updated the readme file.
## Testing recommendations
Compiled and tested on couchdb-sklassen beta first
## Checklist
- [X ] Code is written and works correctly;
- [X ] Changes are covered by tests;
- [X ] Documentation reflects the changes;
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [couchdb-pkg] sklassen commented on issue #68: A template for
compiling under feature/core20
Posted by GitBox <gi...@apache.org>.
sklassen commented on issue #68: A template for compiling under feature/core20
URL: https://github.com/apache/couchdb-pkg/pull/68#issuecomment-610491543
> One question: why? Why revert to building from source? Is it just the directory layout changes?
Fewer dependencies and shorter file paths.
If you can, it is cleaner to build everything from source. The snap is pristine, untainted by external binaries. And it removes a dependency, so every experimental push to an edge channel doesn't need to generate a full .deb file first. In my case, I wanted to change the jiffy version, but I didn't have write access to https://apache.bintray.com/couchdb-deb, but I could point to a local tar-ball.
Another alternative, to avoid the dependency on the tar ball, is to point to git directly. You can also specify a branch as such:
>source: https://github.com/apache/couchdb.git
>source-type: git
>source-branch: 3.x
The purpose of this PR is to show couchdb 3.x can build under core20 with sm60 on the focal and then be ran on older 19.10 and 18.04 installations. Take what you like; discard the rest.
NB. In the snap version couchdb.remsh still requires you to specify the name and cookie, but with a change to the startup script (https://github.com/apache/couchdb/pull/2738) it will respect the ARG_FILE overwrite the same way it does in the couchdb startup script and remsh should work without needing the extra parameters.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [couchdb-pkg] wohali commented on issue #68: A template for
compiling under feature/core20
Posted by GitBox <gi...@apache.org>.
wohali commented on issue #68: A template for compiling under feature/core20
URL: https://github.com/apache/couchdb-pkg/pull/68#issuecomment-610441069
One question: why? Why revert to building from source? Is it just the directory layout changes?
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [couchdb-pkg] wohali commented on issue #68: A template for
compiling under feature/core20
Posted by GitBox <gi...@apache.org>.
wohali commented on issue #68: A template for compiling under feature/core20
URL: https://github.com/apache/couchdb-pkg/pull/68#issuecomment-610516263
> Fewer dependencies
If the packages have unnecessary dependencies, help us fix those in the general sense. That would be useful for the project overall. It doesn't need special casing for just the snap.
If you specifically mean not requiring a package to be built first, see below.
> The snap is pristine, untainted by external binaries.
And yet people ask for these dependencies all the time. For instance, see apache/couchdb#2762 .
> shorter paths
This, I understand.
> If you can, it is cleaner to build everything from source
This is [FUD](https://en.wikipedia.org/wiki/Fear,_uncertainty,_and_doubt) on packages, plain and simple. The packages also build from source. You can even forcibly install them without dependencies if you really want to sort out dependencies yourself (`dpkg -i --force-depends`).
> And it removes a dependency, so every experimental push to an edge channel doesn't need to generate a full .deb file first.
I see it as the exact reverse; it's more mouths for us feed. It's not [DRY](https://en.wikipedia.org/wiki/Don't_repeat_yourself). It means we have to maintain multiple build processes in multiple places and be sure they all stay in sync, even as the process changes. For a project maintainer this is a headache I don't need or want.
> In my case, I wanted to change the jiffy version, but I didn't have write access to https://apache.bintray.com/couchdb-deb, but I could point to a local tar-ball.
For us, this isn't a consideration. We only release "official" binaries from actual tagged versions, per Apache policy. We can't (and won't) release any custom builds that aren't directly from a tag or tarball.
>The purpose of this PR is to show couchdb 3.x can build under core20 with sm60 on the focal and then be ran on older 19.10 and 18.04 installations
For this to land, the PR needs to have its conflicts resolved. If it cannot be merged without apache/couchdb#2738 then it will also block on that.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services