Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Baruch Siach <baruch@tkos.co.il>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH 1/1] encfs: new package
Date: Wed, 29 Aug 2018 12:01:25 +0300	[thread overview]
Message-ID: <87a7p5y2ai.fsf@tkos.co.il> (raw)
In-Reply-To: <1535532457.29191.12.camel@tkos.co.il>

Hi Yair,

Yair Ben Avraham writes:
> I will post a second version of the patch. Two questions below,
>
> On Wed, 2018-08-29 at 09:13 +0300, Baruch Siach wrote:
>> > +ENCFS_VERSION = 1.9.5
>> > +ENCFS_SOURCE = encfs-$(ENCFS_VERSION).zip
>> > +ENCFS_SITE = $(call github,vgough,encfs,v$(ENCFS_VERSION))
>> Upstream provides a proper tarball:
>> 
>>  https://github.com/vgough/encfs/releases/download/v1.9.5/encfs-1.9.
>> 5.tar.gz
>> 
>> Please use that instead of the github helper.
>
> tarball is prior to zip archive with github helper?

tarballs are generally preferred over using the github helper. We only
use the github helper when there is no upstream provided tarball. In
this case the tarball is even more preferable since it is signed by the
upstream maintainer.

>> > +-DRUNTIME_OUTPUT_DIRECTORY=$(BUILD_DIR) \
>> > +-DCMAKE_RUNTIME_OUTPUT_DIRECTORY=$(BUILD_DIR)/usr/bin \
>> These are only used for the bundled libxml2. If you use the Buildroot
>> package, this would not be needed.
>
> If not using the above *RUNTIME_OUTPUT_DIRECTORY variables, I'm getting
> during linking the CXX executables:
> arm-buildroot-linux-uclibcgnueabihf/bin/ld: cannot open output file
> encfs: Is a directory

OK. Please add a comment about that. Would also be nice to report
upstream. I'm far from a cmake expert, but it looks like a bug.

baruch

-- 
     http://baruch.siach.name/blog/                  ~. .~   Tk Open Systems
=}------------------------------------------------ooO--U--Ooo------------{=
   - baruch at tkos.co.il - tel: +972.52.368.4656, http://www.tkos.co.il -

      reply	other threads:[~2018-08-29  9:01 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-08-29  4:32 [Buildroot] [PATCH 1/1] encfs: new package Yair Ben Avraham
2018-08-29  6:13 ` Baruch Siach
2018-08-29  8:47   ` Yair Ben Avraham
2018-08-29  9:01     ` Baruch Siach [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=87a7p5y2ai.fsf@tkos.co.il \
    --to=baruch@tkos.co.il \
    --cc=buildroot@busybox.net \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox