Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 637518 - app-editors/scite-4.0.2 - Tidy ebuild
Summary: app-editors/scite-4.0.2 - Tidy ebuild
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal
Assignee: Jonas Stein
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2017-11-14 19:51 UTC by Chris Mayo
Modified: 2017-11-16 22:40 UTC (History)
2 users (show)

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


Attachments
tidied ebuild scite-4.0.2.ebuild (scite-4.0.2.ebuild,2.64 KB, text/plain)
2017-11-15 08:30 UTC, Ervin Peters
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Chris Mayo 2017-11-14 19:51:59 UTC
Various suggestions for improving the ebuild:

- eutils.eclass not needed
- one sed not two for gtk/makefile
- consistent sed paths and error messages
- Encoding now removed from desktop file
- ensure variable declared in src_compile is local
- dodir's in src_install not needed

Patch:

https://github.com/gentoo/gentoo/pull/6190
Comment 1 Ervin Peters 2017-11-15 08:30:55 UTC
Created attachment 504274 [details]
tidied ebuild scite-4.0.2.ebuild

thanks for the hints, applied and worksforme.

ervin
Comment 2 Larry the Git Cow gentoo-dev 2017-11-16 22:40:58 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=0d0a3c4df07778110309e721003fb90de4398297

commit 0d0a3c4df07778110309e721003fb90de4398297
Author:     Chris Mayo <aklhfex@gmail.com>
AuthorDate: 2017-11-16 19:27:12 +0000
Commit:     Jonas Stein <jstein@gentoo.org>
CommitDate: 2017-11-16 22:10:20 +0000

    app-editors/scite: Tidy ebuild
    
    - eutils.eclass not needed
    - one sed not two for gtk/makefile
    - consistent sed paths and error messages
    - Encoding now removed from desktop file
    - ensure variable declared in src_compile is local
    - dodir's in src_install not needed
    
    Closes: https://bugs.gentoo.org/637518
    
    Package-Manager: Portage-2.3.13, Repoman-2.3.3

 app-editors/scite/scite-4.0.2.ebuild | 43 +++++++++++++++---------------------
 1 file changed, 18 insertions(+), 25 deletions(-)