Skip to content

A few updates to enable building on Raspberry PI/Ubuntu 16.04 #318

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

Merged
merged 1 commit into from
Oct 25, 2017

Conversation

nettlep
Copy link
Contributor

@nettlep nettlep commented Oct 24, 2017

I ran into some compilation failures while building on the Raspberry PI Model 3. These issues were simply type conversion warnings that resulted in errors (due to -Werror in the compilation settings.)

There may be more related issues, but what is attached is the minimal set of changes required to get it to build for me.

My setup: Raspberry PI 3 Model B

$ uname -a
Linux rpi 4.9.37-v7+ #1017 SMP Thu Jul 13 11:26:04 BST 2017 armv7l armv7l armv7l GNU/Linux

$ lsb_release -a
No LSB modules are available.
Distributor ID: Ubuntu
Description:    Ubuntu 16.04.3 LTS
Release:        16.04
Codename:       xenial

${SRCROOT}/swift/utils/build-script --preset=buildbot_linux_1604 install_destdir="${SRCROOT}/__install"  installable_package="${SRCROOT}/__install.tar.gz"

My apologies for not providing the exact compilation error output. It would take hours to produce those errors. If they are needed, I can restart that process to do so.

src/source.c Outdated
@@ -2023,15 +2023,15 @@ static inline unsigned long
_dispatch_source_timer_data(dispatch_source_t ds, dispatch_unote_t du)
{
dispatch_timer_source_refs_t dr = du._dt;
unsigned long data, prev, clear_prev = 0;
unsigned long long data, prev, clear_prev = 0;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this should be uint64_t

@MadCoder
Copy link
Contributor

This looks good, please squash the branch so that I can merge it.

@MadCoder
Copy link
Contributor

@swift-ci test

@nettlep
Copy link
Contributor Author

nettlep commented Oct 25, 2017

Done, I think? Let me know if I didn't do that right.

@MadCoder
Copy link
Contributor

Nope it says 4 commits, should be just the one

@MadCoder
Copy link
Contributor

  1. git fetch
  2. git rebase -i origin master
  3. leave pick of your original commit there and prefix the second one with fixup drop any other line
  4. exit editor and force-push

@nettlep
Copy link
Contributor Author

nettlep commented Oct 25, 2017

Okay, the 'git rebase -i origin master' provided just a single 'noop' entry. So I rebased like normal but used your pick/fixup instruction and it looks like that got it right.

Sorry 'bout this. It's not your job to teach git. Appreciate it, though.

@MadCoder MadCoder merged commit 3b06f54 into swiftlang:master Oct 25, 2017
ktopley-apple pushed a commit that referenced this pull request Dec 6, 2018
A few updates to enable building on Raspberry PI/Ubuntu 16.04

Signed-off-by: Daniel A. Steffen <dsteffen@apple.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants