-
Notifications
You must be signed in to change notification settings - Fork 25
Add 3.9 Packaging Scripts and Cross Compilation Support #7
Add 3.9 Packaging Scripts and Cross Compilation Support #7
Conversation
scripts/ci/debian/debian/files
Outdated
| @@ -0,0 +1,1044 @@ | |||
| /home/parallels/llvm/build/mono-llvm-3.9/usr | |||
| /home/parallels/llvm/build/mono-llvm-3.9/usr/lib | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Are these dir names correct ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No, they're not. I'm not convinced that this file is necessary since we build on the bots with this as it is. This whole .deb configuration is a bit of a hack right now. I'm using helper tools that seem to want special-snowflake configuration. Right now we just copy this directory over. This is definitely not the right way to do it, but it works. Most of my attention was on getting llvm itself working on linux. I put this up here to ask @directhex what route he thinks I should take to make this configuration step more sane.
06f9129 to
e2dcd60
Compare
scripts/ci/debian/control
Outdated
| Source: mono-llvm | ||
| Priority: optional | ||
| Maintainer: Alex Kyte <alkyte@microsoft.com> | ||
| Build-Depends: debhelper (>= 8.0.0) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This line is where you specify which packages are required to build - for example, the upstream LLVM 3.9 package's build depends are here: http://sources.debian.net/src/llvm-toolchain-snapshot/1:3.9~svn274438-1/debian/control/#L6-L11
If you specify this correctly, you can generate a "good" source package which can be passed through to the chroot package builder, which makes building on multiple architectures WAY easier.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As in debian/compat note, this should be lowered to debhelper (>= 7.0.50~) for now.
e2dcd60 to
16f6701
Compare
102f9da to
3c92d8e
Compare
3c92d8e to
f2d44fa
Compare
|
I can't push to |
No description provided.