Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

--- Comment #26 from Dominik 'Rathann' Mierzejewski <>  
2009-01-20 15:40:41 EDT ---
Some more comments:

Requires: libX11 >= 1.1.4 

Why do you need a specific libX11 version? Please add a comment with an

Also, please sort both Requires: and BuildRequires: alphabetically for better

 FlexDock is a Java docking framework for use in cross-platform
 Swing applications

Why is the description indented? Also, it's missing the end period.

#Modify the jni dir that is hardcoded in the patch
cp -pf %{PATCH0} %{PATCH0}.tmp
sed -i 's!%%{_libdir}/%%{name}/!%{_libdir}/%{name}/!' %{PATCH0}

Why don't you just modify the patch?

  if [ `uname -m` == "x86_64" ] || [ `uname -m` == "ppc64" ] ; then
   JDK_DIR=`echo %{_jdkdir} | sed 's!/$!!'`.`uname -m` 

I suggest using a case statement here. It'll be easy to add other arches later.
For example sparc64.

# Apache commons Logging component
# ./lib/commons-logging-1.1.jar
rm -f ./lib/commons-logging-1.1.jar
ln -s %{_javadir}/commons-logging.jar ./lib/commons-logging-1.1.jar

#remove jmf, as it is only used in a demo,
#which is unused after patching
rm -rf ./lib/jmf/

#" Looks" project
rm -f ./lib/looks-2.1.1.jar
ln -s %{_javadir}/jgoodies-looks.jar ./lib/looks-2.2.1.jar

#skinlf "Skin look and Feel" project
rm -f ./lib/skinlf.jar
ln -s %{_javadir}/skinlf.jar ./lib/skinlf.jar

Manually making symlinks is fragile, especially for versioned JARs. It'll break
when their version changes. I think you should use build-jar-repository for

 dos2unix -d2u $i;

What does -d2u option do? It's not documented in --help or in the manpage.

flexdock has funny arch flags, such as "" on i386
SOFILE=`find ./ -name libRubberBand*so`

Couldn't you just patch the build system not to put arch in lib filename?

strip --strip-unneeded $SOFILE

Calling strip from the specfile is not allowed. rpm post-build scripts do that
while building the debuginfo package.

%post -p /sbin/ldconfig

%postun -p /sbin/ldconfig

These are not needed, as the library is not in search path (and not
directly linkable).

*Tue Jan 20 2009 <mycae(a!t)> 0.5.1-8
- Set defattr
- Fix arch (again)
- Change install dir from %%libdir to %%libdir/%%name 
- Update patch0 to match changed so dir + make dynamic
*Sat Dec 20 2008  <mycae(a!t)> 0.5.1-7
- Re-enable system skinlf link & jar check.

Changelog entries must have a space between * and date.

Configure bugmail:
------- You are receiving this mail because: -------
You are on the CC list for the bug.

Fedora-package-review mailing list

Reply via email to