search

postmodern / chruby

Changes the current Ruby
MIT License
2.86k stars 190 forks source link

`test/opt/rubies` is in the way of building a package. #334

Open x-yuri opened 9 years ago

x-yuri commented 9 years ago

I've managed to build a package for debian recently. So, firstly I wanted to report that the aforementioned directory is in the way of building the package (or rather creating a source package). To make it work I created a patch, which adds rm -rf test/opts/rubies to the clean target. So if you backport it to chruby, it would make it slightly easier to create a debian package.

Secondly, I built the package, following debian's howto's (mostly this one), not with fpm. There might be some minor issues, but I believe it's mostly ready for uploading to debian archive. But I don't have time to maintain the package. So if that makes sense, I can share what I've got for now. (Should I do this in separate issue with more fitting name?) And I'm okay with answering the questions, if any. And if I know the answers, apparently. That is, I'm willing to help someone maintain the package :)

postmodern commented 9 years ago

test/opt/rubies is only created by the test/setup script, which is ran before the test suite. test/ should be omitted from any package.

I'd love to host the debian package files in a deb/ directory to compliant the rpm/ directory.

x-yuri commented 9 years ago

I was surely not trying to put tests into debian package. The tests are run when building the package, so the next time you try to build the package (after making some changes, supposedly), the build fails because make clean leaves tests/opt/rubies as is, doesn't clean everything. For now, I fix it on "debian side", but we can backport it to upstream. Here that is.

As for building debian package I guess I'll share what I've got here in a while, and we'll decide what to do with it.