[opensuse-factory] New package twilio-utils
Hello, I'd like to submit a new package to factory, twilio-utils in network:telephony. This is a set of command line scripts for interacting with the Twilio: http://www.twilio.com/ Twilio is an Internet telephony and SMS gateway service. The twilio-utils project contains a few UNIX command-line utilities that are handy when working with Twilio: * sendsms - Send an outbound SMS message via Twilio * smslen - Calculate SMS payload length * twilog - Download Twilio notifications into syslog https://build.opensuse.org/package/show?package=twilio-utils&project=network... The upstream project is also maintained by me. Thanks, -Archie -- Archie L. Cobbs -- To unsubscribe, e-mail: opensuse-factory+unsubscribe@opensuse.org To contact the owner, e-mail: opensuse-factory+owner@opensuse.org
On Tue, May 21, 2013 at 12:38 PM, Archie Cobbs <archie@dellroad.org> wrote:
Looks like you're ready for step 7: http://lizards.opensuse.org/2010/03/23/how-to-add-new-packages-to-the-opensu... feedback in advance of that to hopefully save you a reject: === Add an initial version number to the changes file" https://build.opensuse.org/package/view_file?expand=1&file=twilio-utils.chan... === In your spec file you have: install %{buildroot}%{_datadir}/doc/packages/%{name}/twilio.conf.sample %{buildroot}%{defaultconf} ... %doc %{_datadir}/doc/packages/%{name} That's wrong. %doc is a macro that automatically copies files to that directory, so if twilio.conf is in the top dir of your tarball you just need: %doc twilio.conf.sample === You're missing the equivalent of %doc COPYING Having a copy of the license in %{_datadir}/doc/packages/%{name} is mandatory I do believe. === You have: %defattr(0644,root,root,0755) %attr(0755,root,root) %{_bindir}/* %attr(0644,root,root) %{_mandir}/man1/* That is normally: %defattr(-,root,root,-) %{_bindir}/* %{_mandir}/man1/* You should only have to force the mod bits if something is wrong in the make install step. Since you're upstream, I assume you're setting the mod bits right in make install. fyi: If you want to discuss your specfile more, post to opensuse-packaging. Specfile syntax is not really on-topic here. Greg Greg -- To unsubscribe, e-mail: opensuse-factory+unsubscribe@opensuse.org To contact the owner, e-mail: opensuse-factory+owner@opensuse.org
On Tue, May 21, 2013 at 12:38 PM, Archie Cobbs <archie@dellroad.org> wrote:
Looks like you're ready for step 7: http://lizards.opensuse.org/2010/03/23/how-to-add-new-packages-to-the-opensu... feedback in advance of that to hopefully save you a reject: === Add an initial version number to the changes file" https://build.opensuse.org/package/view_file?expand=1&file=twilio-utils.chan... === In your spec file you have: install %{buildroot}%{_datadir}/doc/packages/%{name}/twilio.conf.sample %{buildroot}%{defaultconf} ... %doc %{_datadir}/doc/packages/%{name} That's wrong. %doc is a macro that automatically copies files to that directory, so if twilio.conf is in the top dir of your tarball you just need: %doc twilio.conf.sample === You're missing the equivalent of %doc COPYING Having a copy of the license in %{_datadir}/doc/packages/%{name} is mandatory I do believe. === You have: %defattr(0644,root,root,0755) %attr(0755,root,root) %{_bindir}/* %attr(0644,root,root) %{_mandir}/man1/* That is normally: %defattr(-,root,root,-) %{_bindir}/* %{_mandir}/man1/* You should only have to force the mod bits if something is wrong in the make install step. Since you're upstream, I assume you're setting the mod bits right in make install. fyi: If you want to discuss your specfile more, post to opensuse-packaging. Specfile syntax is not really on-topic here. Greg Greg -- To unsubscribe, e-mail: opensuse-factory+unsubscribe@opensuse.org To contact the owner, e-mail: opensuse-factory+owner@opensuse.org
participants (2)
-
Archie Cobbs -
Greg Freemyer