107580 – Typo in mplayerplug-in-3.11 ebuild

Bug 107580 - Typo in mplayerplug-in-3.11 ebuild

Summary: Typo in mplayerplug-in-3.11 ebuild

Status:	RESOLVED FIXED

Alias:	None

Product:	Gentoo Linux
Classification:	Unclassified
Component:	Current packages (show other bugs)
Hardware:	All Linux

Importance:	High normal (vote)
Assignee:	Mozilla Gentoo Team

URL:
Whiteboard:
Keywords:

Depends on:
Blocks:	107579
	Show dependency tree

Reported:	2005-09-29 03:15 UTC by vltg0903
Modified:	2005-09-29 07:18 UTC (History)
CC List:	2 users (show)

See Also:
Package list:
Runtime testing required:	---

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description vltg0903 2005-09-29 03:15:50 UTC

In line 31:
if ! use gecko-sdk -> if use gecko-sdk

Reproducible: Always
Steps to Reproduce:
1.
2.
3.

Comment 1 Olliver Schinagl 2005-09-29 05:07:09 UTC

Confirmed. The negation should be removed.

Comment 2 Anders Storsveen 2005-09-29 05:49:31 UTC

Also:

I got:

config.status: creating install.sh
config.status: creating config.h
/usr/portage/net-www/mplayerplug-in/mplayerplug-in-3.11.ebuild: line 38:
--with-gecko-sdk=/usr/lib/gecko-sdk: No such file or directory

!!! ERROR: net-www/mplayerplug-in-3.11 failed.
!!! Function src_compile, Line 38, Exitcode 127
!!! econf failed
!!! If you need support, post the topmost build error, NOT this status message.

In:

    if ! use gecko-sdk; then
        einfo Configuring to build using gecko-sdk
        myconf="${myconf} --with-gecko-sdk=/usr/$(get_libdir)/gecko-sdk"

fi

I'm no expert, the econf thingy is used on the --force-gtk2 option below...

like this:         econf "--with-gecko-sdk=/usr/$(get_libdir)/gecko-sdk"

Comment 3 Rodrigo Severo 2005-09-29 06:28:28 UTC

There is one other correction: join lines 33 and 34 in a single one:

  myconf="${myconf} --with-gecko-sdk=/usr/$(get_libdir)/gecko-sdk"

Yes I know I should send a patch but I changed the orignal ebuild and so I don't
have a reference over which create a patch.

Comment 4 Joe Jezak (RETIRED) gentoo-dev

2005-09-29 07:18:29 UTC

Yeah, sorry, I've fixed this in CVS.