Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!

Bug 429698

Summary: app-misc/freemind: please review prefix changes
Product: Gentoo Linux Reporter: Christoph Junghans (RETIRED) <junghans>
Component: Current packagesAssignee: Java team <java>
Status: RESOLVED FIXED    
Severity: normal CC: prefix
Priority: Normal Keywords: Inclusion, PATCH
Version: unspecified   
Hardware: All   
OS: Linux   
Whiteboard:
Package list:
Runtime testing required: ---
Bug Depends on:    
Bug Blocks: 315803    
Attachments: patch for freemind-0.9.0.ebuild
patch for freemind-0.9.0.ebuild

Description Christoph Junghans (RETIRED) gentoo-dev 2012-08-03 18:36:47 UTC
Created attachment 320200 [details, diff]
patch for freemind-0.9.0.ebuild

Only trivial changes.
Comment 1 Ralph Sennhauser (RETIRED) gentoo-dev 2012-08-03 21:37:14 UTC
freeminds build.xml contains the following code:

        <!-- Mac OS X -->
        <condition property="isMacOs">
            <os family="mac"/>
        </condition>
        <antcall target="dist_icon"/>
        <antcall target="dist_macos"/>

the intention is to create a dmg for use in native OSX. So instead of depending on jarbundler this code shouldn't be run.
Comment 2 Christoph Junghans (RETIRED) gentoo-dev 2012-08-05 19:16:44 UTC
Created attachment 320482 [details, diff]
patch for freemind-0.9.0.ebuild

(In reply to comment #1)
> freeminds build.xml contains the following code:
> 
>         <!-- Mac OS X -->
>         <condition property="isMacOs">
>             <os family="mac"/>
>         </condition>
>         <antcall target="dist_icon"/>
>         <antcall target="dist_macos"/>
> 
> the intention is to create a dmg for use in native OSX. So instead of
> depending on jarbundler this code shouldn't be run.
You are right, patch updated.
Comment 3 Ralph Sennhauser (RETIRED) gentoo-dev 2012-08-06 13:53:18 UTC
Fixed, thanks.

*freemind-0.9.0-r1 (06 Aug 2012)

  06 Aug 2012; Ralph Sennhauser <sera@gentoo.org> +freemind-0.9.0-r1.ebuild:
  Drop jsr173, it's not a dep, it just happened to work around a broken stax
  virtual.
  Thanks to Christoph Junghans <ottxor@gentoo.org> for the Prefix fixes.
  #429698