Don't hardcode number of processors in Linux build instructions (PR #24700)

27 views
Skip to first unread message

Blake-Madden

unread,
Jul 12, 2024, 2:25:40 PMJul 12
to wx-...@googlegroups.com, Subscribed

Saying $(nproc) works fine.


You can view, comment on, or merge this pull request online at:

  https://github.com/wxWidgets/wxWidgets/pull/24700

Commit Summary

  • cb20764 Don't hardcode number of processors in build instructions

File Changes

(1 file)

Patch Links:


Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.Message ID: <wxWidgets/wxWidgets/pull/24700@github.com>

Lauri Nurmi

unread,
Jul 14, 2024, 1:39:45 PMJul 14
to wx-...@googlegroups.com, Subscribed

Saying $(nproc) works fine.

It works fine (even on most non-Linux *nix nowadays), but on the other hand it also complicates the simple example.


Reply to this email directly, view it on GitHub, or unsubscribe.

You are receiving this because you are subscribed to this thread.Message ID: <wxWidgets/wxWidgets/pull/24700/c2227424208@github.com>

VZ

unread,
Jul 14, 2024, 7:59:17 PMJul 14
to wx-...@googlegroups.com, Subscribed

Yes, I'm not sure if it's worth it, we already mention nproc just above, after all.

Any other opinions?


Reply to this email directly, view it on GitHub, or unsubscribe.

You are receiving this because you are subscribed to this thread.Message ID: <wxWidgets/wxWidgets/pull/24700/c2227525816@github.com>

Blake-Madden

unread,
Jul 24, 2024, 2:04:54 PMJul 24
to wx-...@googlegroups.com, Subscribed

Closed #24700.


Reply to this email directly, view it on GitHub, or unsubscribe.

You are receiving this because you are subscribed to this thread.Message ID: <wxWidgets/wxWidgets/pull/24700/issue_event/13634512403@github.com>

Blake-Madden

unread,
Jul 24, 2024, 2:04:54 PMJul 24
to wx-...@googlegroups.com, Subscribed

OK, we can just leave this as-is.


Reply to this email directly, view it on GitHub, or unsubscribe.

You are receiving this because you are subscribed to this thread.Message ID: <wxWidgets/wxWidgets/pull/24700/c2248615847@github.com>

Reply all
Reply to author
Forward
0 new messages