* [Buildroot] [Bug 4171] New: makedevs-unused-but-set-variable.patch
@ 2011-09-09 23:17 bugzilla at busybox.net
2011-10-09 21:27 ` [Buildroot] [Bug 4171] makedevs-unused-but-set-variable.patch bugzilla at busybox.net
2011-10-23 17:50 ` bugzilla at busybox.net
0 siblings, 2 replies; 3+ messages in thread
From: bugzilla at busybox.net @ 2011-09-09 23:17 UTC (permalink / raw)
To: buildroot
https://bugs.busybox.net/show_bug.cgi?id=4171
Summary: makedevs-unused-but-set-variable.patch
Product: buildroot
Version: 2010.11
Platform: PC
OS/Version: Linux
Status: NEW
Severity: minor
Priority: P5
Component: Other
AssignedTo: unassigned at buildroot.uclibc.org
ReportedBy: hector.oron at gmail.com
CC: buildroot at uclibc.org
Estimated Hours: 0.0
Hello,
makedevs package in buildroot fails to build when `-Wunused-but-set-variable'
is set in the compiler. The following patch attempts to fix this issue:
Index: buildroot-2010.11/package/makedevs/makedevs.c
===================================================================
--- buildroot-2010.11.orig/package/makedevs/makedevs.c 2011-09-09
22:33:32.948124813 +0100
+++ buildroot-2010.11/package/makedevs/makedevs.c 2011-09-09
22:33:55.044124824 +0100
@@ -531,5 +531,5 @@
if (system("/bin/sync"))
bb_error_msg("sync failed, continuing anyway");
- return 0;
+ return ret;
}
Best regards,
-- Hector Oron
--
Configure bugmail: https://bugs.busybox.net/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.
^ permalink raw reply [flat|nested] 3+ messages in thread* [Buildroot] [Bug 4171] makedevs-unused-but-set-variable.patch 2011-09-09 23:17 [Buildroot] [Bug 4171] New: makedevs-unused-but-set-variable.patch bugzilla at busybox.net @ 2011-10-09 21:27 ` bugzilla at busybox.net 2011-10-23 17:50 ` bugzilla at busybox.net 1 sibling, 0 replies; 3+ messages in thread From: bugzilla at busybox.net @ 2011-10-09 21:27 UTC (permalink / raw) To: buildroot https://bugs.busybox.net/show_bug.cgi?id=4171 Peter Korsgaard <jacmet@uclibc.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |RESOLVED Resolution| |FIXED --- Comment #1 from Peter Korsgaard <jacmet@uclibc.org> --- Thanks. I had to apply manually as your patch was against an old BR version (/bin/sync call got removed back in March). I guess that's also the reason why you brought this up as we previously built makedevs with -Werror, which we don't do any more. -- Configure bugmail: https://bugs.busybox.net/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug. ^ permalink raw reply [flat|nested] 3+ messages in thread
* [Buildroot] [Bug 4171] makedevs-unused-but-set-variable.patch 2011-09-09 23:17 [Buildroot] [Bug 4171] New: makedevs-unused-but-set-variable.patch bugzilla at busybox.net 2011-10-09 21:27 ` [Buildroot] [Bug 4171] makedevs-unused-but-set-variable.patch bugzilla at busybox.net @ 2011-10-23 17:50 ` bugzilla at busybox.net 1 sibling, 0 replies; 3+ messages in thread From: bugzilla at busybox.net @ 2011-10-23 17:50 UTC (permalink / raw) To: buildroot https://bugs.busybox.net/show_bug.cgi?id=4171 Danomi Mocelopolis <d_mo1234@yahoo.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |d_mo1234 at yahoo.com --- Comment #2 from Danomi Mocelopolis <d_mo1234@yahoo.com> 2011-10-23 17:50:41 UTC --- *** Bug 4375 has been marked as a duplicate of this bug. *** -- Configure bugmail: https://bugs.busybox.net/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug. ^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2011-10-23 17:50 UTC | newest] Thread overview: 3+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2011-09-09 23:17 [Buildroot] [Bug 4171] New: makedevs-unused-but-set-variable.patch bugzilla at busybox.net 2011-10-09 21:27 ` [Buildroot] [Bug 4171] makedevs-unused-but-set-variable.patch bugzilla at busybox.net 2011-10-23 17:50 ` bugzilla at busybox.net
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox