-
Notifications
You must be signed in to change notification settings - Fork 21
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
Add preliminary RPM support #4
base: master
Are you sure you want to change the base?
Conversation
…at formula files include common_formula. (This makes it easier to extend brew2deb to build other types of packages)
@cparedes: There's a point in one of the build scripts (%install section via fpm/templates/rpm.erb) that deletes CWD when attempting to re/create one of the build directories. It probably just needs to
( Disposable CentOS 6.2 x86_64 vm - system ruby, rubygems, git, ruby-devel & development tools group) Update: for grins created a build user, same error. |
Simplest fix: Latest fpm master works like a charm. [build@hostname pkg]$ sudo rpm -Uvh libcredis-0.2.3+github1-1.x86_64.rpm
Preparing... ########################################### [100%]
1:libcredis ########################################### [100%]
[build@hostname pkg]$ rpm -ql libcredis
/usr/include/credis.h
/usr/lib/libcredis.a
/usr/lib/libcredis.so
/usr/share/doc/libcredis/README |
brew2deb making RPM's? WHAT?! :)
Could probably cherry pick a few of these changes (notably, could possibly remove credis-rpmtest, but it's a good test to see that it mostly works, so far.)
Had to split apart the libraries in lib/ to separate files, so it's easier to extend (if there's any need for other package managers.) Also -- added more require statements in the brew2deb binary.