Hi,
Yes, now I understand that this was a bad decision for resolving this clang
error. Sorry for not understanding it from the beginning.
Could you please take a look at my new patch for this issue?
2014-06-22 15:12 GMT+04:00 Debian Bug Tracking System <ow...@bugs.debian.org
>:
> This is an automatic notification regarding your Bug report
> which was filed against the src:cdebootstrap package:
>
> #751630: cdebootstrap: FTBFS with clang instead of gcc
>
> It has been closed by Bastian Blank <wa...@debian.org>.
>
> Their explanation is attached below along with your original report.
> If this explanation is unsatisfactory and you have not received a
> better one in a separate message then please contact Bastian Blank <
> wa...@debian.org> by
> replying to this email.
>
>
> --
> 751630: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=751630
> Debian Bug Tracking System
> Contact ow...@bugs.debian.org with problems
>
>
> ---------- Пересылаемое сообщение ----------
> From: Bastian Blank <wa...@debian.org>
> To: 751630-d...@bugs.debian.org
> Cc:
> Date: Sun, 22 Jun 2014 13:09:57 +0200
> Subject: Re: Bug#751630: cdebootstrap: FTBFS with clang instead of gcc
> On Sat, Jun 14, 2014 at 05:38:42PM -0400, Alexander Ovchinnikov wrote:
> > Using the rebuild infrastructure, your package fails to build with clang
> > (instead of gcc).
> > We detected this kinf of error:
> > http://clang.debian.net/status.php?version=3.4&key=UNUSED_FUNCTION
>
> This is a false positive. This function is used in three locations:
>
> > int frontend_progress_set (int n) __attribute__ ((alias ("unused")));
> > int frontend_progress_start (int max) __attribute__ ((alias
> ("unused")));
> > int frontend_progress_step (int step) __attribute__ ((alias
> ("unused")));
>
> Also the patch is crap. Maybe an __attribute__ ((unused)) would be
> okay.
>
> Bastian
>
> --
> You're too beautiful to ignore. Too much woman.
> -- Kirk to Yeoman Rand, "The Enemy Within", stardate
> unknown
>
> ---------- Пересылаемое сообщение ----------
> From: Alexander Ovchinnikov <sanek23...@gmail.com>
> To: Debian Bug Tracking System <sub...@bugs.debian.org>
> Cc:
> Date: Sat, 14 Jun 2014 17:38:42 -0400
> Subject: cdebootstrap: FTBFS with clang instead of gcc
> Source: cdebootstrap
> Severity: minor
> Tags: patch
> User: pkg-llvm-t...@lists.alioth.debian.org
> Usertags: clang-ftbfs
>
> Hello,
>
> Using the rebuild infrastructure, your package fails to build with clang
> (instead of gcc).
>
> We detected this kinf of error:
> http://clang.debian.net/status.php?version=3.4&key=UNUSED_FUNCTION
>
> Full build log is available here:
>
> http://clang.debian.net/logs/2014-01-14/cdebootstrap_0.5.10_unstable_clang.log
>
> Thanks,
> Alexander
>
>
> -- System Information:
> Debian Release: jessie/sid
> APT prefers unstable
> APT policy: (500, 'unstable')
> Architecture: amd64 (x86_64)
>
> Kernel: Linux 3.13-1-amd64 (SMP w/1 CPU core)
> Locale: LANG=en_US.UTF-8, LC_CTYPE=en_US.UTF-8 (charmap=UTF-8)
> Shell: /bin/sh linked to /bin/dash
>
>
--
Alexander
Best regards.
--- cdebootstrap-0.5.10/src/frontend/standalone/message.c 2012-05-17
17:15:04.000000000 -0400
+++ cdebootstrap-0.5.10-my/src/frontend/standalone/message.c 2014-06-22
16:15:33.905660249 -0400
@@ -34,6 +34,8 @@
#include "log.h"
#include "message.h"
+static int unused () __attribute__ ((unused));
+
enum message_level message_level = MESSAGE_LEVEL_NORMAL;
void frontend_log_message (log_message_name message_name, va_list args)
@@ -76,7 +78,7 @@
fflush (out);
}
-static int unused ()
+int unused ()
{
return 0;
}
