Bug#819072: closed by Sylvestre Ledru <sylvestre at debian.org>

[Sylvestre Ledru]

Hello

This is probably bug 795310
a patch would be welcome :)
Thanks,
S

Le 21/07/2016 à 16:20, mikey d a écrit :
> On Tue, 5 Jul 2016 09:53:03 -0400 Brad King <brad.king at kitware.com <mailto:brad.king at kitware.com>> wrote:
>> On 07/02/2016 03:54 PM, Debian Bug Tracking System wrote:
>> > It has been closed by Sylvestre Ledru <sylvestre at debian.org <mailto:sylvestre at debian.org>>.
>>
>> I can confirm 3.8.1-3 works now.  Thanks!
>>
>> However, it looks like 3.9 needs additional work due to more upstream
>> changes.  The changes needed to support it will likely be incompatible
>> with the 3.8 packages so you may need a new svn branch in the packaging
>> repo for 3.9.  I'll follow up in
>>
>>  https://llvm.org/bugs/show_bug.cgi?id=28325
>>
>> with more details.
>>
>> -Brad
>>
>>
>>
>
>
> It appears that with 3.8.1-4, there is an issue with help2man and how BUILD_DIR is handled.
>
> "help2man: can't get `--help' info from build-llvm//bin/llvm-mc"
>
> The following code sets this early on:
>
> ifeq (${AUTOCONF_BUILD},yes)
>
> BUILD_DIR=Release
>
> endif
>
> However, later on, this is referenced during man page generation:
>
> LD_LIBRARY_PATH=$(DEB_INST)/usr/lib/llvm-$(LLVM_VERSION)/lib/:/usr/lib/*/libfakeroot help2man --no-discard-stderr --version-string=$(LLVM_VERSION) $(TARGET_BUILD)/$(BUILD_DIR)/bin/$$f > debian/man/$$f-$(LLVM_VERSION).1;
>
> What this does then, is just mess up this process, adding two slashes. The path should be be corrected here, maybe in any of the two ways:
>
> 1. Adjust the setting of BUILD_DIR, accounting for both cases (yes and no)
>
> 2. Use a sed expression ahead of help2man like:
>
>
> ifeq (${AUTOCONF_BUILD},yes)
>
> VERSION_STRING=$(LLVM_VERSION) $(TARGET_BUILD)/$(BUILD_DIR)
>
> else
>
> VERSION_STRING=$(LLVM_VERSION) $(TARGET_BUILD)
>
> endif
>
>
> Then later:
>
> LD_LIBRARY_PATH=$(DEB_INST)/usr/lib/llvm-$(LLVM_VERSION)/lib/:/usr/lib/*/libfakeroot help2man --no-discard-stderr --version-string=$(VERSION_STRING)/bin/$$f > debian/man/$$f-$(LLVM_VERSION).1;
>
> Going to test this out myself, but the build takes quite a while.
>
>
>
>
> --
>
> Michael DeGuzis
> Email: mdeguzis at gmail.com <mailto:mdeguzis at gmail.com>
> Website: http://www.libregeek.org <http://www.libregeek.org/>
> Linked In Resume/Profile <http://www.linkedin.com/profile/view?id=124915388&trk=nav_responsive_tab_profile>
> Projects: GitHub Projects <https://github.com/ProfessorKaos64>
>
>
> _______________________________________________
> Pkg-llvm-team mailing list
> Pkg-llvm-team at lists.alioth.debian.org
> http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/pkg-llvm-team
>