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
waf: enable cross-compilation support #1397
Conversation
@cpaelzer iirc you wrote this plugin originally, mind taking a quick look? |
]) | ||
return env | ||
|
||
def enable_cross_compilation(self): |
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.
Something still missing - if so drop this and line 160.
Otherwise I think a comment why it is there for people like me would be very kind.
Other than the comment above LGTM, I have nothing around to test it. |
@@ -14,10 +14,25 @@ | |||
# You should have received a copy of the GNU General Public License | |||
# along with this program. If not, see <http://www.gnu.org/licenses/>. | |||
|
|||
import os | |||
|
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.
why two spaces?
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.
Thanks @kalikiana! But well, this also need the steps in manual_tests.md, so we don't forget about it.
CC and CXX environment variables need to be set accordingly to cross-compile with Waf.