Added debian package configuration and dockerfile for building#5
Added debian package configuration and dockerfile for building#5hafkensite wants to merge 1 commit intoossobv:masterfrom
Conversation
wdoekes
left a comment
wdoekes
left a comment
There was a problem hiding this comment.
Prima idee van de build-met-docker wat mij betreft.
Moet nog wel wat met versienummers fixen. En een paar andere kleine fixes.
En een kort stukje readme dat in elk geval verklaart wat al de docker-turds in de repo doen, of natuurlijk uitlegt hoe je een deb-package build.
| @@ -0,0 +1,32 @@ | |||
| # Object files | |||
There was a problem hiding this comment.
Hierboven even een "# When using docker in a build context, a COPY need not copy build products." ofzo.
Ik snapte niet meteen waar de dockerignore voor was.
| *.x86_64 | ||
| *.hex | ||
|
|
||
| #debian build artifacts |
| Source: rtpsniff | ||
| Section: net | ||
| Priority: optional | ||
| Maintainer: OSSO B.V. <walter+rtpsniff@osso.nl> |
| Architecture: any | ||
| Depends: ${shlibs:Depends}, ${misc:Depends} | ||
| Description: <insert up to 60 chars description> | ||
| <insert long description, indented with spaces> |
| Source: <https://github.com/ossobv/rtpsniff> | ||
|
|
||
| Files: * | ||
| Copyright: 2008,2009,2014 OSSO B.V. <walter+rtpsniff@osso.nl> |
| Files: * | ||
| Copyright: 2008,2009,2014 OSSO B.V. <walter+rtpsniff@osso.nl> | ||
| License: <special license> | ||
| License: GPL-3.0 |
There was a problem hiding this comment.
Special license? Niet gewoon alleen GPL-3.0?
| # See debhelper(7) (uncomment to enable) | ||
| # output every command that modifies files on the build system. | ||
| #DH_VERBOSE = 1 | ||
| export MOD_OUT=out_syslog |
There was a problem hiding this comment.
Hierboven even een "# Select syslog as output module."
| %: | ||
| dh $@ --with systemd | ||
|
|
||
| override_dh_usrlocal: |
|
|
||
| [Service] | ||
| User=root | ||
| ExecStart=/usr/local/sbin/rtpsniff eth0 100 "udp and not port 53" |
There was a problem hiding this comment.
Local? Voor deb files willen we dit gewoon in /usr/sbin.
| ADD . /rtpsniff-1.0 | ||
| WORKDIR /rtpsniff-1.0 | ||
| RUN dpkg-buildpackage -uc -us -b | ||
| CMD cp ../rtpsniff_1.0-1_amd64.deb . |
There was a problem hiding this comment.
Ik heb een beetje twijfels bij het zo hardcoden van de versienummers. We krijgen straks voor elke build een 1.0 en een changelog met 1.0-1.
Misschien iets met envvars doen? Zie ook build-asterisk shellscript in 80e30a9171159716dbf78f3025eb0e259b0bdc02 van asterdebvg repo? Tevens is daarbij nog een kort stukje readme over hoe je moet builden.
wdoekes
force-pushed
the
master
branch
4 times, most recently
from
8196933 to
c091bff
Compare
2 participants
No description provided.