Bug 439265 - Review Request: libnb-platform7 - NetBeans Platform 6.0
Summary: Review Request: libnb-platform7 - NetBeans Platform 6.0
Status: CLOSED DUPLICATE of bug 456337
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
Target Milestone: ---
Assignee: Nobody's working on this, feel free to take it
QA Contact: Fedora Extras Quality Assurance
Depends On: 439262 439263
TreeView+ depends on / blocked
Reported: 2008-03-27 19:03 UTC by Jaroslav Tulach
Modified: 2008-08-05 16:24 UTC (History)
7 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Last Closed: 2008-08-05 16:24:35 UTC

Attachments (Terms of Use)

Description Jaroslav Tulach 2008-03-27 19:03:19 UTC
Spec URL: http://www.xelfi.cz/download/dev/SRPMS/netbeans-platform.spec
SRPM URL: http://www.xelfi.cz/download/dev/SRPMS/libnb-platform7-6.0.1-2.src.rpm

NetBeans Platform is a framework for development of
rich client Swing applications. It contains powerful
module system and a set of modules providing various
functionalities needed for simplification of
development of modular desktop applications.

Comment 1 Colin Walters 2008-03-27 19:27:52 UTC
On an overall level, does Fedora really have all the current Netbeans
dependencies?  The Build-Depends is pretty short.  Does Netbeans ship a copy of
any libraries in source form even, not necessarily jar?

%setup -q -c
find . -type d | xargs -t chmod 755
find . -type f -exec chmod 644 {} ";"
find . -type f \( -iname "*.jar" -o -iname "*.zip" \) | xargs -t %{__rm} -f

Longer term, I'd try to have netbeans generate a clean source tarball/zip rather
than relying on this.

RMF="%{__rm} -rf"
INS="%{__cp} -r"

A lot of this stuff seems like it could be put into the upstream build system.

%dir %{clusterdir}/platform7/

That's it?  There's no /usr/bin/netbeans?  You need to have a .desktop file at
least too.

Comment 2 Colin Walters 2008-03-27 19:35:00 UTC
Oh, I missed that this is just the -plaftorm.  Ignore the last comment.

Comment 3 Jaroslav Tulach 2008-03-27 19:58:42 UTC
The dependencies for the platform are java > 1.6.0, javahelp2 and swing-layout

"A lot of this stuff seems like..." - is it a blocker? If it builds, then it 
builds, and that is the most important goal, imho. Btw. The reason you see 
these tricks with environment variables is that we want to share as much as 
possible with the debian packages and these variables represent differences 
between the scripts for RPM based distros and debian.

Comment 4 Colin Walters 2008-03-27 20:08:21 UTC
"A lot of this stuff seems like..." - is it a blocker? 

Nope, not a blocker; we have plenty of spec files with much hairier rules.  Just
a suggestion.

Comment 5 Colin Walters 2008-03-27 20:26:56 UTC
Looking at the upstream source, I see a lot of empty directories with names of
other projects:

drwxr-xr-x mslama/mslama       0 2008-01-29 11:46
-rw-r--r-- mslama/mslama    2169 2007-09-26 16:48

etc.  Do any these get downloaded and built during 'make build'?  Be aware that
the Fedora build system (koji) has no Internet access.  The right thing to do is
get everything packaged as separate RPM components - this ensures that we only
need to ship one new package when there is a security or reliability issue, as
well as helping reduce duplication of code upstream.

Comment 6 Jaroslav Tulach 2008-03-27 23:43:27 UTC
Yes, we use separate RPMs for each component (like javahelp2, lucene2 - only 
for the IDE, atd.). No, we do not need internet connection. This is 
experimentally verified, these SRPMS were successfully built on Mandriva's 
cooker and I expect their build system to have similar restrictions.

Btw. if we resolve issues with the platform and manage to build it on koji, 
I'll provide SRPMS for IDE and other libraries. This platform case is here to 
estimate how much work that will be.

Comment 7 Colin Walters 2008-03-29 15:47:48 UTC
When I try to build with:

mock -r fedora-devel-x86_64 libnb-platform7-6.0.1-2.src.rpm

I get this error in build.log:

error: line 3: Tag takes single token only: Release:	%mkrel 2

I'm not familiar with %mkrel; is it from another RPM system?

Comment 8 Jaroslav Tulach 2008-03-29 19:36:22 UTC
Re: "%mkrel 2" - yes, it has been told me to use that by mandriva guys, I 
lived in an impression that it is some standard makro that resolves to 2mdv on 
mandriva and 2fc9 on fedora. If you tell me what to use instead, I can use 

On the other hand it would not be bad if you defined %mkrel somewhere 
globally, but I can imagine it would have bigger impact and required more 
reviews than only including this package.

Comment 9 Peter Lemenkov 2008-03-31 17:43:32 UTC
The package's name differs from upstream - current name, libnb-platform7, is
quite ugly. Why not to name it netbeans-platform?

Comment 10 Peter Lemenkov 2008-03-31 17:52:45 UTC
Few notes:

* You should drop epoch field completely. 
* BuildRoot should be changed to something like

BuildRoot:      %{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)

* You should use %{dist} tag instead of %mkrel

Release:        2%{?dist}

* Drop macro %source_top - it's not used.
* Drop macro %section - it's not used.
* I don't understand the purpose of %clusterdir macro - why not to substitute
%{_datadir}/netbeans ?

In any case - great work! Thanks.

Are you planning to push NetBeans to Fedora?

Comment 11 Jaroslav Tulach 2008-03-31 21:37:34 UTC
OK to almost all your comments. I'll change the spec.

Re. "Are you planning to push NetBeans to Fedora?" - yes, that is what I'd 
like to! Now tell me what I need to do to make that happen? If I attach 
modified spec, will somebody apply it? Or will I get access to your version 
control & build system to do it myself? Which of these and when?

Comment 12 Colin Walters 2008-04-01 03:05:25 UTC
Basically we iterate the spec, once it's ready there is a final review.  At that
point you will get a Fedora account, a CVS directory and you can make changes
and initiate builds.

Comment 13 Colin Walters 2008-04-01 03:43:33 UTC
So yes, please post a updated .spec and SRPM files after the fixes above.

Comment 14 Jaroslav Tulach 2008-04-02 13:43:54 UTC

Spec URL: http://www.xelfi.cz/download/dev/SRPMS/netbeans-platform.spec

Things that I have not changed: %{clusterdir} - it is easier to move NetBeans 
around, and anyway it is just local variable, not visible from outside.

The name libnb-platform7 signals two things: 1. this is just a library, 
nothing to be executed. It is version 7 of the platform, when we release new 
incompatible version we make it 8.

Comment 15 Colin Walters 2008-04-28 15:57:14 UTC
This has a dependency on java-rpmbuild that isn't in Fedora, I think we ran into
that somewhere else?

But we need javahelp2, so I'll take a look at #439263 first.

Comment 16 Colin Walters 2008-04-28 22:39:30 UTC
Hi Jaroslav, I noticed that you've also submitted this to JPackage:

We should probably do the fixes there and then once it's done synchronize with

Comment 17 Jaroslav Tulach 2008-04-30 12:40:00 UTC
I working with jpackage simplifies something for me, then OK. Anyway I have 
changed the javahelp2 package, which was the last blocker for this package, as 
far as I remember. So maybe you could just check the latest submission for 

Comment 18 Thomas Fitzsimmons 2008-06-26 20:03:18 UTC
Jaroslav, please remove the java-rpmbuild build requirement (or propose
java-rpmbuild for inclusion in Fedora, if appropriate) so that this review can

Comment 19 Joshua Sumali 2008-07-21 15:28:53 UTC
Has there been any more progress with this package?

Comment 20 Alexei Mokeev 2008-07-23 11:05:50 UTC
Review Request for NetBeans Platform 6.1:

Comment 21 Lillian Angel 2008-08-05 16:24:35 UTC

*** This bug has been marked as a duplicate of bug 456337 ***

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