Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 266402 - [EBUILD] www-client/midori-0.1.8 another ebuild
Summary: [EBUILD] www-client/midori-0.1.8 another ebuild
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: High enhancement (vote)
Assignee: Markus Ullmann (RETIRED)
URL: http://software.twotoasts.de/index.ph...
Whiteboard:
Keywords: EBUILD
: 276257 (view as bug list)
Depends on:
Blocks:
 
Reported: 2009-04-16 18:27 UTC by Ivan Anishchuk
Modified: 2009-07-31 07:51 UTC (History)
12 users (show)

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


Attachments
ebuild for midori-0.1.5 (midori-0.1.5.ebuild,1.27 KB, text/plain)
2009-04-16 18:28 UTC, Ivan Anishchuk
Details
ebuild for midori-0.1.6 (midori-0.1.6.ebuild,1.59 KB, text/plain)
2009-04-26 21:19 UTC, Ivan Anishchuk
Details
ebuild for midori-0.1.6 (midori-0.1.6.ebuild,1.61 KB, text/plain)
2009-04-26 21:34 UTC, Ivan Anishchuk
Details
reverse trash menu order (midori-0.1.6-history_menu_order.patch,7.01 KB, patch)
2009-04-27 01:20 UTC, Ivan Anishchuk
Details | Diff
ebuild for midori-0.1.6 (midori-0.1.6-r1.ebuild,1.62 KB, text/plain)
2009-04-27 01:22 UTC, Ivan Anishchuk
Details
git ebuild for midori (midori-9999.ebuild,1.56 KB, text/plain)
2009-04-27 01:25 UTC, Ivan Anishchuk
Details
git ebuild for midori (midori-9999.ebuild,1.56 KB, text/plain)
2009-04-27 12:41 UTC, Ivan Anishchuk
Details
ebuild for midori-0.1.6 (midori-0.1.6-r1.ebuild,1.62 KB, text/plain)
2009-04-30 09:35 UTC, Ivan Anishchuk
Details
git ebuild for midori (midori-9999.ebuild,1.56 KB, text/plain)
2009-04-30 09:36 UTC, Ivan Anishchuk
Details
ebuild for midori-0.1.6 (midori-0.1.6-r1.ebuild,2.12 KB, text/plain)
2009-05-19 15:42 UTC, Ivan Anishchuk
Details
git ebuild for midori (midori-9999.ebuild,2.09 KB, text/plain)
2009-05-19 15:44 UTC, Ivan Anishchuk
Details
ebuild for midori-0.1.7 (midori-0.1.7-r1.ebuild,2.25 KB, text/plain)
2009-05-30 09:38 UTC, Ivan Anishchuk
Details
reverse trash menu order (midori-0.1.7-history_menu_order.patch,7.01 KB, patch)
2009-05-30 09:40 UTC, Ivan Anishchuk
Details | Diff
git ebuild for midori (midori-9999.ebuild,2.22 KB, text/plain)
2009-05-30 10:00 UTC, Ivan Anishchuk
Details
ebuild for midori-0.1.7 (midori-0.1.7-r1.ebuild,2.27 KB, text/plain)
2009-06-01 19:38 UTC, Ivan Anishchuk
Details
fit ebuild for midori (midori-9999.ebuild,2.24 KB, text/plain)
2009-06-01 19:43 UTC, Ivan Anishchuk
Details
git ebuild for midori (midori-9999.ebuild,2.24 KB, text/plain)
2009-06-15 10:11 UTC, Ivan Anishchuk
Details
ebuild for midori-0.1.7 (midori-0.1.7-r1.ebuild,2.27 KB, text/plain)
2009-06-15 10:21 UTC, Ivan Anishchuk
Details
updated midori 0.1.7 ebuild (midori-0.1.7-r2.ebuild,2.28 KB, text/plain)
2009-07-11 15:17 UTC, Peter Read
Details
ebuild for midori-0.1.7 (midori-0.1.7-r1.ebuild,2.26 KB, text/plain)
2009-07-16 05:23 UTC, Ivan Anishchuk
Details
git ebuild for midori (midori-9999.ebuild,2.23 KB, text/plain)
2009-07-16 05:25 UTC, Ivan Anishchuk
Details
git ebuild for midori (midori-9999.ebuild,2.26 KB, text/plain)
2009-07-18 14:25 UTC, Ivan Anishchuk
Details
ebuild for midori-0.1.7 (midori-0.1.7-r1.ebuild,2.29 KB, text/plain)
2009-07-18 14:26 UTC, Ivan Anishchuk
Details
ebuild for midori-0.1.7 (midori-0.1.7-r1.ebuild,2.29 KB, text/plain)
2009-07-18 14:38 UTC, Ivan Anishchuk
Details
git ebuild for midori (midori-9999.ebuild,2.26 KB, text/plain)
2009-07-18 14:39 UTC, Ivan Anishchuk
Details
ebuild for midori-0.1.8 (midori-0.1.8.ebuild,2.26 KB, text/plain)
2009-07-21 10:56 UTC, Ivan Anishchuk
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Ivan Anishchuk 2009-04-16 18:27:06 UTC
This is another ebuild for midori-0.1.5. It supports 'idn' & 'unique' use flags, use 'soup' has been removed because now midori require libsoup hard (not optional). webkit-gtk-0_p42000 has been replaced with net-libs/webkit-gtk-1.1.1

Reproducible: Always
Comment 1 Ivan Anishchuk 2009-04-16 18:28:56 UTC
Created attachment 188595 [details]
ebuild for midori-0.1.5
Comment 2 Mart Raudsepp gentoo-dev 2009-04-17 11:58:37 UTC
I think we need webkit-gtk-1.1.5 in the tree then too alongside the SVN ones or replacing them.
Due to that personally interested too (would do the webkit-gtk bits if had more time, maybe I shall take that time over the weekend).
Then the SVN builds, if any, would be webkit-gtk-1.1.5_p200904nn, etc for proper ordering
Comment 3 Ivan Anishchuk 2009-04-17 20:51:14 UTC
(In reply to comment #2)

it will be nice :)
Comment 4 Ivan Anishchuk 2009-04-26 21:19:29 UTC
Created attachment 189563 [details]
ebuild for midori-0.1.6

+ Version bump
+ Use-flag 'doc' for api-doc building has been added
* Documentation path fixed (/usr/share/doc/midori -> /usr/share/doc/midori-0.1.6)
* Some minor changes
Comment 5 Ivan Anishchuk 2009-04-26 21:34:08 UTC
Created attachment 189567 [details]
ebuild for midori-0.1.6

+ depend on dev-python/docutils for building user help
Comment 6 Ivan Anishchuk 2009-04-27 01:20:34 UTC
Created attachment 189580 [details, diff]
reverse trash menu order

This patch reverses trash menu order to proper: the latest closed tab is on top.
Comment 7 Ivan Anishchuk 2009-04-27 01:22:06 UTC
Created attachment 189581 [details]
ebuild for midori-0.1.6

Ebuild with trash order patch
Comment 8 Ivan Anishchuk 2009-04-27 01:25:14 UTC
Created attachment 189583 [details]
git ebuild for midori

Live git ebuild, like previous
Comment 9 Ivan Anishchuk 2009-04-27 12:41:36 UTC
Created attachment 189612 [details]
git ebuild for midori

cosmetic fix
Comment 10 Ivan Anishchuk 2009-04-30 09:35:02 UTC
Created attachment 189939 [details]
ebuild for midori-0.1.6

* fix path for user-doc ( /usr/share/doc/${P} -> /usr/share/doc/${PF} )
Comment 11 Ivan Anishchuk 2009-04-30 09:36:09 UTC
Created attachment 189940 [details]
git ebuild for midori
Comment 12 Ivan Anishchuk 2009-05-19 15:42:18 UTC
Created attachment 191794 [details]
ebuild for midori-0.1.6

*Now version and git ebuilds have the same structure.
*PATCHES variable with list of patches.
!Take care about version-specific patches. They'll not be applied after ebuild renaming, but...
Comment 13 Ivan Anishchuk 2009-05-19 15:44:37 UTC
Created attachment 191796 [details]
git ebuild for midori

just renamed git ebuild (and without patches instead of release ebuild)
Comment 14 Ivan Anishchuk 2009-05-30 09:36:57 UTC
0.1.7 version bump and some minor fixes
Comment 15 Ivan Anishchuk 2009-05-30 09:38:45 UTC
Created attachment 192960 [details]
ebuild for midori-0.1.7
Comment 16 Ivan Anishchuk 2009-05-30 09:40:40 UTC
Created attachment 192962 [details, diff]
reverse trash menu order
Comment 17 Ivan Anishchuk 2009-05-30 10:00:03 UTC
Created attachment 192963 [details]
git ebuild for midori
Comment 18 Ivan Anishchuk 2009-06-01 19:38:58 UTC
Created attachment 193174 [details]
ebuild for midori-0.1.7

*fix KEYWORDS
Comment 19 Ivan Anishchuk 2009-06-01 19:43:18 UTC
Created attachment 193177 [details]
fit ebuild for midori
Comment 20 René 'Necoro' Neumann 2009-06-10 22:18:11 UTC
The DEPEND-string looks completely wrong...

libsoup, webkit-gtk, libxml2, gtk+, idn, gettext, sqlite and libunique are definitly also RDEPENDs :) (I doubt, that the midori-build links them staticly :P)
Comment 21 Ivan Anishchuk 2009-06-14 05:02:22 UTC
(In reply to comment #20)
> The DEPEND-string looks completely wrong...
> 
> libsoup, webkit-gtk, libxml2, gtk+, idn, gettext, sqlite and libunique are
> definitly also RDEPENDs :) (I doubt, that the midori-build links them staticly
> :P)
> 

Of course :) . But configure script checks if they're installed and fails if not...
Comment 22 René 'Necoro' Neumann 2009-06-14 10:27:24 UTC
Then add them to both ;). The problem with them only being in DEPEND is, that an "emerge --depclean --with-bdeps=n" will remove them ;)
Comment 23 Ivan Anishchuk 2009-06-15 10:11:37 UTC
Created attachment 194758 [details]
git ebuild for midori

Is this right?
Comment 24 Ivan Anishchuk 2009-06-15 10:21:33 UTC
Created attachment 194759 [details]
ebuild for midori-0.1.7
Comment 25 Walther 2009-07-03 06:15:21 UTC
*** Bug 276257 has been marked as a duplicate of this bug. ***
Comment 26 Walther 2009-07-03 10:21:11 UTC
I tested the midori-0.1.7-r1 ebuild, and it works fine, however, there are two things I must note:

1. Why is "dev-python/docutils" a mandatory dependency? For that matter, why is the config option "userdocs" forced to being selected? I need no docs, and by editing the ebuild I could save myself from installing an additional package for documentation I don't want.

2. Should there be a way to strip MAKE_OPTS from unsupported flags? At first I got an error because "./waf build" does not understand the -s switch (MAKE_OPTS="-j3 -s").
Comment 27 Peter Read 2009-07-11 14:52:47 UTC
I know, I'm impatient...  but is there anything we can help test to get this moving?  and/or is this in any of the o.g.o overlays?  it seems we have a working (albeit maybe not optimal) ebuild...

if not I'll go mess around in my own overlay but that's probably not very helpful beyond my few boxes :)
Comment 28 Peter Read 2009-07-11 15:17:20 UTC
Created attachment 197559 [details]
updated midori 0.1.7 ebuild

this works for me, and makes optional the docutils dependancy (possibly it should be a seperate userdocs use flag, if you like...)
Comment 29 Peter Read 2009-07-11 15:19:20 UTC
just uploaded a very slightly modified ebuild - puts the userdocs under the doc use flag, maybe that's poor judgement on my part but seemed sane to me.  

Also I dropped passing makeopts to waf - that might be contentious I guess, but as it doesn't support or silently drop all make opts and it can cause failures it seems safer to me...
Comment 30 Ivan Anishchuk 2009-07-13 22:45:34 UTC
(In reply to comment #26)
> I tested the midori-0.1.7-r1 ebuild, and it works fine, however, there are two
> things I must note:
> 
> 1. Why is "dev-python/docutils" a mandatory dependency? For that matter, why is
> the config option "userdocs" forced to being selected? I need no docs, and by
> editing the ebuild I could save myself from installing an additional package
> for documentation I don't want.

There's policy to install readme and 'f1'-help files by default. Use-flag 'doc' is for api documentation and other developer docs. There are no standard flags to ommit installing of readme and others... (i.e. all xfce helps are installed by default). About docutils: maybe we need separate use-flag, but i mean that geeks may make theirs own gentoo without docutils and userdocs, and with blackjack and hookers :)
 
> 2. Should there be a way to strip MAKE_OPTS from unsupported flags? At first I
> got an error because "./waf build" does not understand the -s switch
> (MAKE_OPTS="-j3 -s").

It's simplier to drop it. Makeops are not necessary for this package.
Comment 31 Ivan Anishchuk 2009-07-13 22:59:47 UTC
(In reply to comment #27)
> I know, I'm impatient...  but is there anything we can help test to get this
> moving?  and/or is this in any of the o.g.o overlays?  it seems we have a
> working (albeit maybe not optimal) ebuild...

maybe...
 
> if not I'll go mess around in my own overlay but that's probably not very
> helpful beyond my few boxes :)

u can share portage tree with local overlay on network with nfs :)
Comment 32 Ivan Anishchuk 2009-07-13 23:04:15 UTC
(In reply to comment #28)
u must not change version number if there is no code/patches has been updated. userdocs must not be under 'doc' flag. i will upload ebuild with dropped makeopts soon.
Comment 33 Peter Read 2009-07-14 14:52:13 UTC
(In reply to comment #32)
> u must not change version number if there is no code/patches has been updated.
> userdocs must not be under 'doc' flag. i will upload ebuild with dropped
> makeopts soon.
> 

Sure I kinda expected the 'doc' flag to not be ideal, but I think it needs to be optional via USE flag as many don't like pulling in arguably superfluous packages.  

Glad you agree re: dropping the makeopts.
Comment 34 Ivan Anishchuk 2009-07-16 05:18:37 UTC
(In reply to comment #33)
> (In reply to comment #32)
> > userdocs must not be under 'doc' flag. i will upload ebuild with dropped

> Sure I kinda expected the 'doc' flag to not be ideal, but I think it needs to
> be optional via USE flag as many don't like pulling in arguably superfluous
> packages.  
 
> ls -lh /usr/share/doc/midori-9999/user/
> total 16K
> -rw-r--r-- 1 root root 14K 2009-07-12 19:21 midori.html

> emerge -s docutils
> ... 
> *  dev-python/docutils
>     Latest version available: 0.5
>     Latest version installed: 0.5
> ...
>      Size of files: 1,246 kB
> ...

Userdocs is one 13kB html file. Weight docutils'sources is about one MB. Where is a trouble? Why so many clamor about them? Helps and mans are very usable for beginners and must be installed by default. 

Nevertheless there are some possible solutions:

You can sharpen ebuild so as to install really *all* docs with 'dodoc' command and use RESTRICT="doc" (docutils will be installed).

Or we can decide to hide userdocs under use flag (under which??? maybe maintainers know?)

Or we can ask upstream to include generated in release tarballs (ask kalikana if you want :) ). Together with the first it will allow not to install docutils and docs.

Comment 35 Ivan Anishchuk 2009-07-16 05:23:22 UTC
Created attachment 198121 [details]
ebuild for midori-0.1.7

* dropped makeopts
Comment 36 Ivan Anishchuk 2009-07-16 05:25:30 UTC
Created attachment 198123 [details]
git ebuild for midori
Comment 37 Ivan Anishchuk 2009-07-16 05:45:32 UTC
(In reply to comment #28)
> Created an attachment (id=197559) [edit]
> updated midori 0.1.7 ebuild

Please obsolete your attachment.
Comment 38 Walther 2009-07-16 06:02:56 UTC
(In reply to comment #34)
> > ls -lh /usr/share/doc/midori-9999/user/
> > total 16K
> > -rw-r--r-- 1 root root 14K 2009-07-12 19:21 midori.html
> 
> > emerge -s docutils
> > ... 
> > *  dev-python/docutils
> >     Latest version available: 0.5
> >     Latest version installed: 0.5
> > ...
> >      Size of files: 1,246 kB
> > ...
> 
> Userdocs is one 13kB html file. Weight docutils'sources is about one MB. Where
> is a trouble? Why so many clamor about them? Helps and mans are very usable for
> beginners and must be installed by default. 

Hmm...

> emerge -s midori
 
*  www-client/midori
      Latest version available: 0.1.7-r1
      Latest version installed: 0.1.7-r1
      Size of files: 452 kB

Do I really have to install something that weighs twice the browser for some documentation I don't want? ;P 

It seems a viable solution is to use a local use flag that is enabled by default ("userdoc"?). I do not contend that user documentation must be installed by default, but not giving users the choice to skip it in the ebuild goes against my favourite feature of ebuilds, the ability to choose what components get built in...
Comment 39 Ivan Anishchuk 2009-07-18 14:17:28 UTC
(In reply to comment #38)
> (In reply to comment #34)

> Do I really have to install something that weighs twice the browser for some
> documentation I don't want? ;P 

You have not. You always can do something with it youself :)

> It seems a viable solution is to use a local use flag that is enabled by
> default ("userdoc"?). I do not contend that user documentation must be
> installed by default, but not giving users the choice to skip it in the ebuild
> goes against my favourite feature of ebuilds, the ability to choose what
> components get built in...
 
Okey. I decide to do so. It will be 'html' use-flag (what will think lamers^Wbeginners about this flag for browser?.. :-)) )
Comment 40 Ivan Anishchuk 2009-07-18 14:25:54 UTC
Created attachment 198419 [details]
git ebuild for midori

+ 'html' use-flag for user html documentation.
Comment 41 Ivan Anishchuk 2009-07-18 14:26:53 UTC
Created attachment 198420 [details]
ebuild for midori-0.1.7
Comment 42 Ivan Anishchuk 2009-07-18 14:38:18 UTC
Created attachment 198422 [details]
ebuild for midori-0.1.7

trivial fix
Comment 43 Ivan Anishchuk 2009-07-18 14:39:36 UTC
Created attachment 198423 [details]
git ebuild for midori
Comment 44 Ivan Anishchuk 2009-07-21 10:56:59 UTC
Created attachment 198686 [details]
ebuild for midori-0.1.8

New minor release.

- Old 'reverse trash menu order' patch doesn't work.
Comment 45 Tony Vroon (RETIRED) gentoo-dev 2009-07-27 18:15:17 UTC
I'm sorry to say that modifying the inherit & KEYWORD lines in this fashion would trash the metadata cache. As such, it can not be allowed in the portage tree. This also makes it hard for me to cherry-pick your improvements.
As such, I will consider this a version bump request and resolve the bug as fixed. When you have specific ebuild improvements to suggest, please do so in a new bug.

+*midori-0.1.8 (27 Jul 2009)
+
+  27 Jul 2009; <chainsaw@gentoo.org> +midori-0.1.8.ebuild, metadata.xml:
+  Version bump, as requested by Jan Aniŝĉuk in bug #266402. Taking
+  maintainership together with the XFCE herd, approved by Jokey on IRC.
Comment 46 Ivan Anishchuk 2009-07-29 13:34:30 UTC
(In reply to comment #45)
> I'm sorry to say that modifying the inherit & KEYWORD lines in this fashion
> would trash the metadata cache. As such, it can not be allowed in the portage
> tree. This also makes it hard for me to cherry-pick your improvements.
> As such, I will consider this a version bump request and resolve the bug as
> fixed. When you have specific ebuild improvements to suggest, please do so in a
> new bug.

Hmm... And what about, for example, sys-kernel/genkernel ebuilds? They have the same stucture with inherit.
Comment 47 Mart Raudsepp gentoo-dev 2009-07-29 15:26:57 UTC
I think it might actually be fine, due to $PV probably being available while generating the metadata (it's gotten from the filename of the ebuild).

http://devmanual.gentoo.org/ebuild-writing/using-eclasses/index.html#the-inherit-function
and
http://devmanual.gentoo.org/general-concepts/portage-cache/index.html
confirm that it's fine.
Comment 48 Mart Raudsepp gentoo-dev 2009-07-29 15:29:04 UTC
Anyways, lets take this to a new bug then please for clean improvements as requested by Tony. Please CC me as well there
Comment 49 Ivan Anishchuk 2009-07-31 07:51:15 UTC
(In reply to comment #48)
> Anyways, lets take this to a new bug then please for clean improvements as
> requested by Tony. Please CC me as well there
> 

moved to bug #279768