iksaif / euscan

Ebuild Upstream Scanner
http://euscan.iksaif.net/
GNU General Public License v2.0
15 stars 14 forks source link

Allow to specify and ebuild instead of a package name #4

Closed iksaif closed 11 years ago

iksaif commented 12 years ago

We should be able to run: $ euscan /usr/portage/app-portage/euscan/euscan-0.1.0.ebuild

Should be easy, just add the tree to os.environ['PORTDIR_OVERLAY'] and parse the path.

volpino commented 12 years ago

@058b50556baefeccdd5145d143b13bb622ccd69c

Fixed ;) Not a really clean solution though

iksaif commented 12 years ago

I tried, and it doesn't seems to work (put an ebuild in a directory that is not in your PORTDIR_OVERLAY) to test.

eshowkw seems to use a simpler solution, but I don't really know if it would work in our case. Anyway, we should probably not use Query here, but directly create a Package() from the ebuild path, probably not possible with current gentoolkit code but I'll look at that later...

volpino commented 12 years ago

Sorry, I can't understand the issue. For me it works: if I try with an ebuild in my portdir overlay it works, if it's outside it doesn't. Like this should be ok, or not?

iksaif commented 12 years ago

On Thu, May 24, 2012 at 7:47 PM, Federico Scrinzi reply@reply.github.com wrote:

Sorry, I can't understand the issue. For me it works: if I try with an ebuild in my portdir overlay it works, if it's outside it doesn't. Like this should be ok, or not?

Well, the point of this feature is that it should work even if not in your overlay list ! I'll give it a try tomorow.

Corentin Chary http://xf.iksaif.net

volpino commented 12 years ago

Sorry, I completely misunderstood the task. It probably needs some more complex refactoring, we cannot query portage any more, we should parse the ebuild. I'll look at it

iksaif commented 12 years ago

On Thu, May 24, 2012 at 11:18 PM, Federico Scrinzi reply@reply.github.com wrote:

Sorry, I completely misunderstood the task. It probably needs some more complex refactoring, we cannot query portage any more, we should parse the ebuild. I'll look at it

We should be able to use portage, the code you imported into helpers.py seems to reconfigure portage adding the current ebuild path to PORTDIR_OVERLAY dynamically, but somehow it doesn't work with gentoolkit...

Corentin Chary http://xf.iksaif.net

iksaif commented 12 years ago

On Thu, May 24, 2012 at 11:19 PM, Corentin Chary corentin.chary@gmail.com wrote:

On Thu, May 24, 2012 at 11:18 PM, Federico Scrinzi reply@reply.github.com wrote:

Sorry, I completely misunderstood the task. It probably needs some more complex refactoring, we cannot query portage any more, we should parse the ebuild. I'll look at it

We should be able to use portage, the code you imported into helpers.py seems to reconfigure portage adding the current ebuild path to PORTDIR_OVERLAY dynamically, but somehow it doesn't work with gentoolkit...

I found a way to do it ! pushed !

Corentin Chary http://xf.iksaif.net