First Last Prev Next    No search results available      Search page      Enter new bug
Bug#: 168617
Alias:
Product:
Component:
Status: RESOLVED
Resolution: FIXED
Assigned To: Chris Gianelloni (RETIRED) <wolf31o2@gentoo.org>
Hardware:
OS:
Version:
Priority:
Severity:
Reporter: Tim Harder <radhermit@gmail.com>
Add CC:
CC:
Remove selected CCs
URL:
Summary:
Status Whiteboard:
Keywords:

Filename Description Type Creator Created Size Actions
easytag-2.0.ebuild easytag-2.0.ebuild text/plain Samuli Suominen 2007-03-01 16:14 0000 937 bytes Details
Create a New Attachment (proposed patch, testcase, etc.) View All

Bug 168617 depends on: Show dependency tree
Bug 168617 blocks:
Votes: 0    Show votes for this bug    Vote for this bug

Additional Comments: (this is where you put emerge --info)


Not eligible to see or edit group visibility for this bug.






View Bug Activity   |   Format For Printing   |   XML   |   Clone This Bug


Description:   Opened: 2007-02-27 19:26 0000
First stable version using the GTK 2 interface.

------- Comment #1 From David Watzke 2007-02-27 22:09:16 0000 -------
Just rename the 1.99.13 ebuild to 2.0 - it works, I've just tested it on my
~amd64 system.

------- Comment #2 From Samuli Suominen 2007-03-01 16:14:40 0000 -------
Created an attachment (id=111712) [edit]
easytag-2.0.ebuild

Renaming kind of works but is invalid. 

- There is no --enable/--disable-nls
- gettext dep was missing
- econf allready has die from portage (see ebuild.sh)
- use emake DESTDIR="${D}" install for installing instead of einstall..

And some minor cleanups.

Anyway, here is a working and hopefully correct ebuild.

------- Comment #3 From David Watzke 2007-03-01 17:02:16 0000 -------
(In reply to comment #2)
> use emake DESTDIR="${D}" install for installing instead of einstall..

Why?

------- Comment #4 From David Watzke 2007-03-01 17:12:14 0000 -------
Btw,
dodoc: NEWS does not exist

------- Comment #5 From Samuli Suominen 2007-03-01 17:39:13 0000 -------
(In reply to comment #3)
> (In reply to comment #2)
> > use emake DESTDIR="${D}" install for installing instead of einstall..
> 
> Why?
> 

DESTDIR is an automake variable to set prefix for installed files. So if
Makefile.am supports it, it should be used. einstall is only for those packages
not respecting DESTDIR.

Read
http://www.gnu.org/software/automake/manual/html_mono/automake.html#Install for
starters.

------- Comment #6 From Chris Gianelloni (RETIRED) 2007-03-02 20:08:31 0000 -------
Samuli, do you use this program?  I use it, but very little, so if you're a
more qualified maintainer for it, you can have it.  At any rate, feel free to
commit your ebuild and close this bug.  I'm on vacation for a bit, so I won't
be able to get to this right away.

------- Comment #7 From Samuli Suominen 2007-03-05 13:36:53 0000 -------
(In reply to comment #4)
> Btw,
> dodoc: NEWS does not exist
> 

Noted, thanks..

------- Comment #8 From Samuli Suominen 2007-03-05 13:37:25 0000 -------
(In reply to comment #6)
> Samuli, do you use this program?  I use it, but very little, so if you're a
> more qualified maintainer for it, you can have it.  At any rate, feel free to
> commit your ebuild and close this bug.  I'm on vacation for a bit, so I won't
> be able to get to this right away.
> 

All done.

First Last Prev Next    No search results available      Search page      Enter new bug