* [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