-
Notifications
You must be signed in to change notification settings - Fork 45
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
Definitive Build Process? #111
Comments
I wasn't aware I had issues disabled, thank you for notifying me. I've enabled them now. You're right that the INSTALL file is not up-to-date. Here's a short version of how I suggest building (in Ubuntu), but I wouldn't call it a definitive guide, I may well have forgotten something:
You mentioned dpkg-buildpackage, so I'm not sure if you were actually looking for instructions for making a .deb? |
Thanks @ejurgensen! This is just what I was looking for. I'll try it sometime this week and update this issue with my results. I mentioned dpkg-buildpackage only because my previous process was to build from an existing .deb (as if it were a patch as mentioned in the link above.) But it never felt complete to me. These feel like a more standard process. One question: If I do a standard pull from git, what is the purpose of the step 1? Are these the dependencies? |
Yes, if you pull from git then step 1 is just so you can find the .dsc file with the dependency list. Btw, remember to use libevent 1 (and remove libevent 2). |
Hi @ejurgensen , I finally got around to trying your steps above. I ran into a couple problems that you might be able to help with:
I'm guessing the lack of a makefile is related to the first issue, but I'm at a bit of a loss on how to debug it. I'm hoping some of what I've said jogs your memory from when you last did it. Thanks! |
I've gone through these steps many times, the last time was just a fews days ago, and I've never encountered permission problems. I always use normal user privileges, you should not be required to use sudo for configure. The makefile error is clearly because configure is not working as it should. I don’t have any idea as to why that may be. How did you get the source – was it with git clone https://github.com/ejurgensen/forked-daapd.git? On what platform are you doing the build? |
I forked your repository into my own, then did a git clone of my fork (git@github.com:DeepFriedTwinkie/forked-daapd.git). But, it's the same codebase, as I haven't committed anything back to it. I'm running Ubuntu 12.04 Server on an 64-bit AMD based machine. I can't seem to SSH into my machine right now, so I can't get more information. I'll play with it a bit more tonight and see if I can get the permissions squared away. I'll look through the configure file to see if there is anything odd in there that could be causing the issue. Thanks for responding! |
One thought: From what dir should steps 5-7 be run? From the root of the repo, or from src? autoreconf is recursive (I think) but not sure if the parent matters. |
This is what I do: git clone https://github.com/ejurgensen/forked-daapd.git I use Ubuntu 12.04 too, but not the server edition. Maybe the server edition is more locked down, so ordinary users don't have access to the stuff in /usr, which configure is checking? BTW the configure script is hard to read because it is generated by autoreconf. If you want to see what is being done you can start with configure.in. |
Hello,
I'm looking for decent instructions on how to build forked-daapd. I've done a few updates and have been able to build by ripping apart the DEB package and then using dpkg-buildpackage. (From here: http://www.mmacleod.ca/blog/2012/05/patching-forked-daapd-so-it-actually-works/)
But I'd rather use an approach that allows me to build properly. I'm guessing the original INSTALL file (https://github.com/ejurgensen/forked-daapd/blob/master/INSTALL) is not up-to-date.
@ejurgensen you seem to be the most active contributor. I would have added this issue on your branch, but you have issues disabled. Hoping you see this and can help!
Thanks!!!
The text was updated successfully, but these errors were encountered: