-
Notifications
You must be signed in to change notification settings - Fork 51
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
MODE_NONE on Fedora - fixes and default #31
Conversation
It now works on Fedora 24 rawhide.
Fedora systems do not need the (outdated) Fedora broot, since they are already running Fedora. By using MODE_HOST, users can instead have sugar build run in their environment, which may be more up to date than the broot.
I am getting the following error when I use this branch, I might want to use the original one for now [c0mrad3@localhost sugar-build]$ ./osbuild pull
= Setup the build system =
* Create the python virtualenv
* Install python packages
remote: Counting objects: 979, done.
remote: Total 979 (delta 0), reused 0 (delta 0), pack-reused 979
Receiving objects: 100% (979/979), 170.16 KiB | 55.00 KiB/s, done.
Resolving deltas: 100% (585/585), done.
Command /home/c0mrad3/sugar-build/build/out/sandbox/install/bin/python2.7 -c "import setuptools, tokenize;__file__='/home/c0mrad3/sugar-build/build/out/sandbox/install/build/sourcestamp/setup.py';exec(compile(getattr(tokenize, 'open', open)(__file__).read().replace('\r\n', '\n'), __file__, 'exec'))" install --record /tmp/pip-N5lSw_-record/install-record.txt --single-version-externally-managed --compile --install-headers /home/c0mrad3/sugar-build/build/out/sandbox/install/include/site/python2.7 failed with error code 1 in /home/c0mrad3/sugar-build/build/out/sandbox/install/build/sourcestamp
Storing debug log for failure in /home/c0mrad3/.pip/pip.log
Traceback (most recent call last):
File "./osbuild", line 484, in <module>
if not main():
File "./osbuild", line 463, in main
if not setup(options.command):
File "./osbuild", line 300, in setup
raise e
subprocess.CalledProcessError: Command '['/home/c0mrad3/sugar-build/build/out/sandbox/install/bin/pip', '-q', 'install', 'git+https://github.com/dnarvaez/osbuild', 'sourcestamp']' returned non-zero exit status 1 |
@i5o just brought up the pip error on irc. Seems to be due to an undeclared dep of |
Sorry, it is marked as a dep. But osbuild installs pip packages before the system packages, which is kinda bad since pip needs the system packages. Maybe that should be reversed? |
Also, gcc in case you don't have it.
Anyway, I got |
Package: |
Ok, I've pushed changes that should fix that |
15:58 now I can't build sugar-web There is no mention of "node" or "npm" in osbuild, so I think @i5o is right. |
d78ca71
to
f28bccc
Compare
Just forced pushed some changed. I fixed the issue where it would try to install everything from dnf every single osbuild run. |
Just installed F24 (a live image with XFCE Spin) |
The system asks me if I want to install the packages We should do (to auto install) |
./osbuild pull Will continue with run |
That's all fine right? On Mon, Nov 2, 2015, 1:07 PM Ignacio Rodríguez notifications@github.com
|
Yep, will test in few minutes the run |
Can't build. |
Trying to: and then build |
tadaaaa!
until
http://fpaste.org/286125/64814971/ [tamara@tamara-laptop sugar-build]$ sudo npm install -g volo we should install the npm packages automatically [tamara@tamara-laptop sugar-build]$ ./osbuild build = Building =
|
Is this ok? Can we merge this @i5o? |
This worked in fedora. I will test if this doesnt work support for Ubuntu (sounds stupid, but 2015-11-13 19:14 GMT-03:00, Sam notifications@github.com:
Ignacio Rodríguez |
Something is broken with ubuntu (USing ubuntu 16.04, I can't even pull with But I will merge this one because it's works on fedora. Ignacio Rodríguez 2015-11-13 19:20 GMT-03:00 Ignacio Rodríguez ignacio@sugarlabs.org:
|
Nevermind, just saw tch comment. Here is what I did in ubuntu and got a error
|
I tested it, works fine, but please see the comments I left. |
Did you tested in Ubuntu? |
Only in Fedora 23... Also, strangely I didn't get the volo error while building sugar-web... I used to get that error... In any case maybe we should have a copy (of osbuild [1] in sugarlabs github page. |
Thanks for the review @tchx84, I made the changes and added them as another commit (9690b23) to the branch. Please pull and merge if they're ok! I don't understand what dnarvaez/osbuild is. The commands are already part of this repo https://github.com/sugarlabs/sugar-build/tree/master/build/commands. Is his version for CI testing? Ubuntu seems messed up. Maybe we should get a list of packages like we have for fedora but on ubuntu? |
Yeah, we need to so something similar like this for Ubuntu. |
good idea. Are you running ubuntu now @i5o? If you can find a list of pacakges that you need to install sugar-build in MODE_NONE on ubuntu then we could make a patch. Otherwise, I'll have a go in a vm. Also 16.04? Isn't that for 2016? |
It's the rawhide of Ubuntu :P I don't have 15.10 right now (i'm not sure if I tested this before 2015-11-14 1:50 GMT-03:00, Sam notifications@github.com:
Ignacio Rodríguez |
MODE_NONE on Fedora - fixes and default
This replaces sugarlabs/sugar-docs#78