Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 25 Nov 2014 20:43:43 +0100
From:      Antoine Brodin <antoine@FreeBSD.org>
To:        Marcus von Appen <mva@freebsd.org>
Cc:        "Montgomery-Smith, Stephen" <stephen@missouri.edu>, "gnome@FreeBSD.org" <gnome@freebsd.org>, Janos Dohanics <web@3dresearch.com>
Subject:   Re: /usr/ports/devel/py-gobject3: typo?
Message-ID:  <CAALwa8kxqb9_uEBY%2B_SU=nnX=AoGeJ6g56BNU8xOot6n_qvfxQ@mail.gmail.com>
In-Reply-To: <20141125080248.GA1026@medusa.sysfault.org>
References:  <20140916124035.2217ba67f72bce56e56eb677@3dresearch.com> <546F767B.4070702@missouri.edu> <CAALwa8=Oee7CUB%2Bojme3MVEzzhWjGF1Ru_pEOMHQyzfb68SOww@mail.gmail.com> <CAHNYxxM8ooo7%2BXNUNROoivBx1kg_8sAaGZmN3tsbG5tw%2BqjCug@mail.gmail.com> <CAALwa8kHs4g2WbRqo-mWFbWiHZoczu3RzzAgE1AFk1h3w2LKpg@mail.gmail.com> <20141125080248.GA1026@medusa.sysfault.org>

next in thread | previous in thread | raw e-mail | index | archive | help
--001a11c11480f0de090508b42046
Content-Type: text/plain; charset=UTF-8

On Tue, Nov 25, 2014 at 9:02 AM, Marcus von Appen <mva@freebsd.org> wrote:
> On, Tue Nov 25, 2014, Antoine Brodin wrote:
>
>> On Tue, Nov 25, 2014 at 5:38 AM, Jia-Shiun Li <jiashiun@gmail.com> wrote:
>> > See if this is helpful to you..
>> >
>> > I encountered this in the middle of building gnome3-lite. On my case
>> > py-gobject3 builds fine alone but fails if built as dependency.
>> >
>> > When failing, there is no
>> >   pygobject-3.14.0-py2.7-freebsd-11.0-CURRENT-amd64.egg-info
>> > as requested in install phase, but there does have
>> >   pygobject-3.14.0-pypython2.7-freebsd-11.0-CURRENT-amd64.egg-info
>> > in the directory. looks something messed up PYTHON_VER for plist.
>> >
>> >
>> > The easier way to reproduce is
>> >   cd /usr/ports/accessibility/caribou && make
>> > vs.
>> >   cd /usr/ports/devel/py-gobject3 && make install
>> >
>> > config.log files from both do not differ. It is more likely 'make
>> > install' causing the problem.
>> >
>> >
>> > My temporary workaround is to rename the file as plist required and continue.
>>
>> Hi mva@,
>>
>> Could you have a look at this?
>> I believe that the DEPENDS_ARGS+=PYTHON_VERSION=${PYTHON_VERSION} in
>> python.mk can cause this kind of issue for some autoconf based
>> projects.
>> In python.mk,  PYTHON_VERSION=python2.7, but in some autoconf based
>> projects PYTHON_VERSION=2.7 (${PYTHON} -c "import sys;
>> sys.stdout.write(sys.version[:3])" 2>/dev/null)
>> So the DEPEND_ARGS may pollute the build of dependencies...
>
> Yes, it looks like DEPENDS_ARGS is the culprit here. The conditionals
> within the configure script of pygobject3-common can run into
> situations, where they do not override PYTHON_VERSION properly.
>
> We have two ways around that:
>
> - override PYTHON_VERSION in the individual ports, which break that way
>   - quickly done
>   - has to be done on a port-per-port basis
>   - adds another "hack" to maintain per port
> - rename PYTHON_VERSION in the ports infrastructure to something else
>   - more clean approach(?)
>   - not done quickly
>   - may affect users using PYTHON_VERSION actively
>   - may need other ports to receive additional changes
>   - needs a full exp-run (without isolations, so poudriere is a no-op)
>
> As workaround, I'd go with the first option for now. Depending on how
> the gnome stuff evolves, we may need to look into option two as well.


The attached patch is ugly, but it seems to work.
I agree that long term solution would be to use variable names that
don't pollute ports build in DEPENDS_ARGS.

Cheers,

Antoine

--001a11c11480f0de090508b42046
Content-Type: text/plain; charset=US-ASCII; name="pg3.diff"
Content-Disposition: attachment; filename="pg3.diff"
Content-Transfer-Encoding: base64
X-Attachment-Id: f_i2xo625i1

SW5kZXg6IGRldmVsL3B5LWdvYmplY3QzL01ha2VmaWxlCj09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIGRldmVsL3B5
LWdvYmplY3QzL01ha2VmaWxlCShyZXZpc2lvbiAzNzM0MjQpCisrKyBkZXZlbC9weS1nb2JqZWN0
My9NYWtlZmlsZQkod29ya2luZyBjb3B5KQpAQCAtMTEsNSArMTEsNyBAQAogCiBNQVNURVJESVI9
CSR7LkNVUkRJUn0vLi4vLi4vZGV2ZWwvcHlnb2JqZWN0My1jb21tb24KIFBMSVNUPQkJJHsuQ1VS
RElSfS9wa2ctcGxpc3QKKyMgT3ZlcnJpZGUgUFlUSE9OX1ZFUlNJT04gZnJvbSBERVBFTkRTX0FS
R1MKK01BS0VfQVJHUz0JUFlUSE9OX1ZFUlNJT049JHtQWVRIT05fVkVSfQogCiAuaW5jbHVkZSAi
JHtNQVNURVJESVJ9L01ha2VmaWxlIgo=
--001a11c11480f0de090508b42046--



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAALwa8kxqb9_uEBY%2B_SU=nnX=AoGeJ6g56BNU8xOot6n_qvfxQ>