[Bug 458090] Review Request: LuxRender - Lux Renderer, an unbiased rendering system

2009-01-21 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=458090





--- Comment #12 from Nicolas Chauvet (kwizart)   2009-01-21 
13:16:43 EDT ---
(In reply to comment #11)
> (In reply to comment #10)
> > that's not what will append for real:
> > 1- As the per user ~/.blender directory is already created no more files 
> > will
> > be copied to the user directory.
> 
> Sorry, On eatch start of blender there will be creating symlink from
> ~/.blnder/scripts to /usr/share/blender/scripts.
It depends on where you consider your error is, so as you like: but quoting
blender-wrapper:
When "" if [ ! -d ~/.blender ]; then ""
install -d ~/.blender/scripts
ln -sf /usr/share/blender/scripts/bpydata ~/blender/scripts/bpydata
^^ this part is wrong - and a misunderstanding of what bpydata and
bpydata/config directories are
(namely the content and configuration directories for scripts that are meant to
be user <<>>).
But there is a typo (blender instead of .blender) so it doesn't work anyway.

Then,:
rm -r ~/.blender/scripts/*.p*
Why end-users aren't allowed to have their own scripts ?


> > 2- Nobody wants to write on LuxBlend_0.1.py?. Instead what the plugin 
> > expects
> > is to write in its config directory and in its content directory. (see the
> > readme.txt).
> 
> I can't find your readme.txt file
rpm -ql blender |grep readme.txt


I'm still unsure of the accurate solution, so it is just workinprogress
thoughts.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 458090] Review Request: LuxRender - Lux Renderer, an unbiased rendering system

2009-03-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=458090





--- Comment #13 from Nicolas Chauvet (kwizart)   2009-03-16 
22:19:08 EDT ---

Spec URL:
http://kwizart.fedorapeople.org/SPECS/LuxRender.spec
SRPM URL:
http://kwizart.fedorapeople.org/SRPMS/LuxRender-0.5-3.fc10.src.rpm
Description: Lux Renderer, an unbiased rendering system

http://koji.fedoraproject.org/koji/taskinfo?taskID=1245132

As said in /usr/share/blender/scripts/bpydata/config/readme.txt
"This folder is for automatically saved scripts configuration data."
It means user writable. Nevertheless it remains relative to the blender script
directory which is ~/.blender/scripts. (Thus,
~/.blender/scripts/bpydata/config/* should be user writable along with
./scripts/bpydata/)

This directory (/usr/share/blender/scripts/bpydata/config/ ) can be seen as
where to store the configuration files for each blender plugin to provide the
default configuration. Each files are meant to be copied into
~/.blender/scripts/bpydata/config/ so end-user can tweak their configuration.
This directory aren't meant to be deleted in anycase in the cost of loosing the
user's plugin configuration.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 458090] Review Request: LuxRender - Lux Renderer, an unbiased rendering system

2009-03-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=458090





--- Comment #14 from Nicolas Chauvet (kwizart)   2009-03-16 
22:26:24 EDT ---
There was a typo in the submitted src.rpm:
http://koji.fedoraproject.org/koji/taskinfo?taskID=1245151

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 458090] Review Request: LuxRender - Lux Renderer, an unbiased rendering system

2009-03-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=458090





--- Comment #15 from Nicolas Chauvet (kwizart)   2009-03-16 
22:33:08 EDT ---
Again:
http://koji.fedoraproject.org/koji/taskinfo?taskID=1245169

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 458090] Review Request: LuxRender - Lux Renderer, an unbiased rendering system

2009-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=458090





--- Comment #16 from Jochen Schmitt   2009-05-06 
15:41:15 EDT ---
I would take a look on your source RPM. But It's seems, it is not available on
koji anymore.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 458090] Review Request: LuxRender - Lux Renderer, an unbiased rendering system

2009-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=458090





--- Comment #17 from Nicolas Chauvet (kwizart)   2009-05-06 
16:14:01 EDT ---
(In reply to comment #16)
> I would take a look on your source RPM. But It's seems, it is not available on
> koji anymore.  
Spec URL:
http://kwizart.fedorapeople.org/SPECS/LuxRender.spec
SRPM URL:
http://kwizart.fedorapeople.org/SRPMS/LuxRender-0.5-3.fc10.src.rpm
Description: Lux Renderer, an unbiased rendering system

^^ This one is still valid (0.6 is nearing, but 0.5 is fine also)

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 458090] Review Request: LuxRender - Lux Renderer, an unbiased rendering system

2009-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=458090





--- Comment #18 from Jochen Schmitt   2009-05-06 
16:41:06 EDT ---
I have test the symlink createn in ~/.blender/scripts with installed blender
and blender-freeworld. the accept symlinks was created during the start of
blender.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 458090] Review Request: LuxRender - Lux Renderer, an unbiased rendering system

2009-05-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=458090





--- Comment #19 from Nicolas Chauvet (kwizart)   2009-05-06 
17:05:03 EDT ---
(In reply to comment #18)
> I have test the symlink createn in ~/.blender/scripts with installed blender
> and blender-freeworld. the accept symlinks was created during the start of
> blender.  

I'm not sure to understand well...
Anyway, would you take this test case:
- Install blender (or the other variant).
- Run blender to ensure the user script directory got created
- Install LuxRender
- Check that the LuxRender scripts ends in the end-user directory.
- Verify that the LuxRender exports scripts are available from the blender
window, and that end-user can tweak LuxRender configuration (which must persist
on next blender run).

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 458090] Review Request: LuxRender - Lux Renderer, an unbiased rendering system

2009-05-07 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=458090





--- Comment #20 from Jochen Schmitt   2009-05-07 
12:56:50 EDT ---
Sorry, You have to install blender and TuxRender first.

If you run blender after install both packages the ~/.blener/script directory
and all symlinks will been created.

I have taken a look on my installation and could found the Texblender export
menu entry.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 458090] Review Request: LuxRender - Lux Renderer, an unbiased rendering system

2009-05-07 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=458090





--- Comment #21 from Nicolas Chauvet (kwizart)   2009-05-07 
13:04:31 EDT ---
(In reply to comment #20)
> Sorry, You have to install blender and TuxRender first.
> 
> If you run blender after install both packages the ~/.blener/script directory
> and all symlinks will been created.
Right, and that's "the" problem... What upstream think about that ?
Of course, most blender plugin extensions will be installed next to blender is
run. the blender plugin directory isn't meant to be used only by blender, but
plugins such as LuxRender should be allowed to store per-user configuration
files...

> I have taken a look on my installation and could found the Texblender export
> menu entry.  
right but once blender is first run when LuxRender was previously installed...

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 458090] Review Request: LuxRender - Lux Renderer, an unbiased rendering system

2009-05-07 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=458090


Jochen Schmitt  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|nob...@fedoraproject.org|joc...@herr-schmitt.de
   Flag||fedora-review?




--- Comment #22 from Jochen Schmitt   2009-05-07 
13:54:46 EDT ---
Good:
+ Base name of the SPEC files matches with package name.
+ Package name fits with the naming convention
* URI tag shows on proper project homepage
* Could downloads sources with spectool -g
+ Sources in Source package matches with upstream
(md5sum: 02174b85917b68ac5f681a4f117b7bd  LuxRender_v05_Source.zip
 7cb0920c7c77ebaa5d7a721be124a27f LuxBlend_v05_Blender_Exporter.zip)
* Package contains several subpackage
+ Package contains valid license tag
+ License Tag contains GPLv3+ and BSD as valid OpenSource licenses
+ Upstream sources contains verbatins copy of the licenses
+ License note in source files seems to mach license tag
+ Consistently usage of rpm macros
+ Proper BuildRoot tag
+ Buildroot will be cleaned on the start of %install and %clean
+ Rpmlint is silent on source RPM.
+ RPM_OPT_FLAGS will honoured.
+ Debuginfo package contains sources
+ Koji scratch build works fine.
* Local build works fine.
* Local install and uninstall works fine.
+ Start of the application works fine
+ Menu entry is ok
+ %files stanza contains no duplicates
+ Package contains proper %Changelog entries


TODO
- Please set blender requirement to bleder >= 2.48a-21 to
  make sure, that the script are available in a corret was.
- Why you add an Req rom the devel-docs package to the main
  Package.
- Please moveove content of the %doc stanza of the libs subpackage
  to the main package
- Rpmlint complaints on binary RPMs:
pmlint LuxRender-*
LuxRender.x86_64: W: no-documentation
LuxRender-blender.x86_64: W: no-documentation
LuxRender-blender.x86_64: E: wrong-script-interpreter
/usr/share/blender/scripts/LuxBlend_0.1.py "BPY"
OK.: Blender specific
LuxRender-blender.x86_64: E: non-executable-script
/usr/share/blender/scripts/LuxBlend_0.1.py 0644
Should be fixed.
LuxRender-blender.x86_64: E: wrong-script-end-of-line-encoding
/usr/share/blender/scripts/LuxBlend_0.1.py
Should be fixed.
LuxRender-core.x86_64: W: no-documentation
OK.
LuxRender-devel.x86_64: W: no-documentation
OK.
LuxRender-lib.x86_64: W: shared-lib-calls-exit /usr/lib64/liblux.so.1.0
e...@glibc_2.2.5
May be OK.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 458090] Review Request: LuxRender - Lux Renderer, an unbiased rendering system

2009-05-07 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=458090





--- Comment #23 from Jochen Schmitt   2009-05-07 
13:58:07 EDT ---
(In reply to comment #21)
> (In reply to comment #20)
> > Sorry, You have to install blender and TuxRender first.
> > 
> > If you run blender after install both packages the ~/.blener/script 
> > directory
> > and all symlinks will been created.
> Right, and that's "the" problem... What upstream think about that ?

Which upstream?

> Of course, most blender plugin extensions will be installed next to blender is
> run. the blender plugin directory isn't meant to be used only by blender, but
> plugins such as LuxRender should be allowed to store per-user configuration
> files...

???

> 
> > I have taken a look on my installation and could found the Texblender export
> > menu entry.  
> right but once blender is first run when LuxRender was previously 
> installed... 

Yes, Yout must install LuxRender first, before you can see this entries. What
is your issue?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 458090] Review Request: LuxRender - Lux Renderer, an unbiased rendering system

2009-05-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=458090





--- Comment #24 from Nicolas Chauvet (kwizart)   2009-05-13 
11:56:19 EDT ---
(In reply to comment #22)
...
> TODO
> - Please set blender requirement to bleder >= 2.48a-21 to
>   make sure, that the script are available in a corret was.
You contradict yourself with this demand:

Either you consider that blender isn't installed and having unversioned
requires will be enought to bring the lastest blender release, either blender
is already installed despite not up2date, and we chances are hight that the
LuxRender-blender files will never be copied to the end-user directory anyway.

As a consequence, using a versioned requires will introduce another problem:
LuxRender works pretty well with blender version < 2.48a.
I would like to avoid the introduction of other problem.

> - Why you add an Req rom the devel-docs package to the main
>   Package.
Thx for having noticed, I've removed it

> - Please moveove content of the %doc stanza of the libs subpackage
>   to the main package
The original
> - Rpmlint complaints on binary RPMs:
> pmlint LuxRender-*
> LuxRender.x86_64: W: no-documentation
> LuxRender-blender.x86_64: W: no-documentation
> LuxRender-blender.x86_64: E: wrong-script-interpreter
> /usr/share/blender/scripts/LuxBlend_0.1.py "BPY"
> OK.: Blender specific
> LuxRender-blender.x86_64: E: non-executable-script
> /usr/share/blender/scripts/LuxBlend_0.1.py 0644
> Should be fixed.
Not necessarily, it depends upon the said script is meant to be sourced or
executed directly.If you look at /usr/lib64/python*/, you will see that very
few files are set +x (probably by misstake actually)

> LuxRender-blender.x86_64: E: wrong-script-end-of-line-encoding
> /usr/share/blender/scripts/LuxBlend_0.1.py
> Should be fixed.
Fixed, thx for having noticed.


Upload in Progress...

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 458090] Review Request: LuxRender - Lux Renderer, an unbiased rendering system

2009-05-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=458090





--- Comment #25 from Nicolas Chauvet (kwizart)   2009-05-13 
12:00:05 EDT ---
> - Please moveove content of the %doc stanza of the libs subpackage
>   to the main package
The original reason was that -lib subpackage will be installed anyway.
But it can be moved to the main also.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 458090] Review Request: LuxRender - Lux Renderer, an unbiased rendering system

2009-05-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=458090





--- Comment #26 from Nicolas Chauvet (kwizart)   2009-05-13 
12:06:01 EDT ---
Spec URL:
http://kwizart.fedorapeople.org/SPECS/LuxRender.spec
SRPM URL:
http://kwizart.fedorapeople.org/SRPMS/LuxRender-0.5-4.fc10.src.rpm
Description: Lux Renderer, an unbiased rendering system

Changelog
- Move %%doc from -lib to main
- Remove Requires main from -devel-docs
- Disable wrong-end-of-line-encoding for LuxBlend_0.1.py

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 458090] Review Request: LuxRender - Lux Renderer, an unbiased rendering system

2009-05-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=458090





--- Comment #27 from Nicolas Chauvet (kwizart)   2009-05-13 
12:35:11 EDT ---
for the blender plugin directory and the plugin installation problem.
I think the solution is to symlinks every files present from
/usr/share/blender/scripts/*.py* to the ~/.blender/scripts directory
unconditionaly when blender is run.
Then, each files from /usr/share/blender/scripts/bpydata/config/* but not
present in the ~/.blender/scripts/bpydata/config/ directory need to be copied
to the end-user directory
(file aren't expected to be copied from /usr/share/blender/scripts/bpydata/*
since they are meant to be temporary files.)

Unfortunately, I'm not sure what Blender.Get("datadir") is. It probably doesn't
match our datadir, specially as it may be used for architecture dependent
plugin (thinking about the yafaray case). But I wonder if it wouldn't have been
easier to take theses:

%{python_sitelib}/blender/
%if %{_lib} == "lib64"
%{python_sitearch}/blender/
%endif

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 458090] Review Request: LuxRender - Lux Renderer, an unbiased rendering system

2009-05-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=458090





--- Comment #28 from Jochen Schmitt   2009-05-13 
13:54:33 EDT ---
(In reply to comment #27)
> for the blender plugin directory and the plugin installation problem.
> I think the solution is to symlinks every files present from
> /usr/share/blender/scripts/*.py* to the ~/.blender/scripts directory
> unconditionaly when blender is run.

This should be done by the current blender-wrapper script in the devel branch.

> Then, each files from /usr/share/blender/scripts/bpydata/config/* but not
> present in the ~/.blender/scripts/bpydata/config/ directory need to be copied
> to the end-user directory
> (file aren't expected to be copied from /usr/share/blender/scripts/bpydata/*
> since they are meant to be temporary files.)

Should be done in 2.48a-22
> 
> Unfortunately, I'm not sure what Blender.Get("datadir") is. It probably 
> doesn't
> match our datadir, specially as it may be used for architecture dependent
> plugin (thinking about the yafaray case). But I wonder if it wouldn't have 
> been
> easier to take theses:
> 
> %{python_sitelib}/blender/
> %if %{_lib} == "lib64"
> %{python_sitearch}/blender/
> %endif  

This should no be an issue.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 458090] Review Request: LuxRender - Lux Renderer, an unbiased rendering system

2009-05-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=458090





--- Comment #29 from Jochen Schmitt   2009-05-13 
13:58:37 EDT ---
(In reply to comment #24)
> (In reply to comment #22)
> ...
> > TODO
> > - Please set blender requirement to bleder >= 2.48a-21 to
> >   make sure, that the script are available in a corret was.
> You contradict yourself with this demand:
> 
> Either you consider that blender isn't installed and having unversioned
> requires will be enought to bring the lastest blender release, either blender
> is already installed despite not up2date, and we chances are hight that the
> LuxRender-blender files will never be copied to the end-user directory anyway.
> 
> As a consequence, using a versioned requires will introduce another problem:
> LuxRender works pretty well with blender version < 2.48a.
> I would like to avoid the introduction of other problem.
> 

Regarding to the last comment pleas set the Req. to blender >= 2.48a-22. This
release or a later release will be introduced  on F-11 and F-10 after APPROVING
of this review

Please keep in mind, that belder-2.49 is upcomming soom.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 458090] Review Request: LuxRender - Lux Renderer, an unbiased rendering system

2009-05-14 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=458090





--- Comment #30 from Nicolas Chauvet (kwizart)   2009-05-14 
08:07:16 EDT ---
>From the blender -22 commit, it should be okay now, so : Wonderfull !
Okay, I've added blender >= 2.48a-22 to the -blender sub package !
(Can I avoid a re-upload ?)

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 458090] Review Request: LuxRender - Lux Renderer, an unbiased rendering system

2009-05-14 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=458090


Jochen Schmitt  changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Comment #31 from Jochen Schmitt   2009-05-14 
11:06:35 EDT ---
OK, As far as I can see, all is OK, so you are APPROVED.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 458090] Review Request: LuxRender - Lux Renderer, an unbiased rendering system

2009-05-14 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=458090


Nicolas Chauvet (kwizart)  changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Comment #32 from Nicolas Chauvet (kwizart)   2009-05-14 
11:33:16 EDT ---
New Package CVS Request
===
Package Name: LuxRender
Short Description: Lux Renderer, an unbiased rendering system
Owners: kwizart
Branches: F-11 F-10 EL-5
Cvsextras Commits: yes

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 458090] Review Request: LuxRender - Lux Renderer, an unbiased rendering system

2009-05-15 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=458090


Kevin Fenzi  changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Comment #33 from Kevin Fenzi   2009-05-15 19:45:29 EDT ---
cvs done.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 458090] Review Request: LuxRender - Lux Renderer, an unbiased rendering system

2009-05-17 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=458090





--- Comment #34 from Jochen Schmitt   2009-05-17 
14:46:32 EDT ---
I have build the -22 release for F-11 and F-10. It will be nice, if you can
bundeled it which your update request.

At least I want to offer you the co-maintainership for blender if you want.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 458090] Review Request: LuxRender - Lux Renderer, an unbiased rendering system

2009-05-18 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=458090


Nicolas Chauvet (kwizart)  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




--- Comment #35 from Nicolas Chauvet (kwizart)   2009-05-18 
07:46:11 EDT ---
(In reply to comment #34)
> I have build the -22 release for F-11 and F-10. It will be nice, if you can
> bundeled it which your update request.
OK, I've just imported LuxRender.
> 
> At least I want to offer you the co-maintainership for blender if you want.  
Thx, I accept.

I guess I need to be approved on the blender acl in order to request an update.
So I will update both blender/LuxRender then.


Thx for this review.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 458090] Review Request: LuxRender - Lux Renderer, an unbiased rendering system

2009-01-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=458090





--- Comment #3 from Nicolas Chauvet (kwizart)   2009-01-20 
10:28:54 EDT ---
Spec URL:
http://kwizart.fedorapeople.org/SPECS/LuxRender.spec
SRPM URL:
http://kwizart.fedorapeople.org/SRPMS/LuxRender-0.5-2.fc10.src.rpm
Description: Lux Renderer, an unbiased rendering system

Koji build on Rawhide
http://koji.fedoraproject.org/koji/taskinfo?taskID=1068814

Changelog
- Fix compilation with boost 1.34 - (backport)
- Subpackage the Blender_Exporter plugin 
- Subpackage -devel-docs for doxygen docs.

Note, despite been checked with cmake, the libXdamage-devel aren't needed. I
will remove the dependency before import.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 458090] Review Request: LuxRender - Lux Renderer, an unbiased rendering system

2009-01-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=458090


Jochen Schmitt  changed:

   What|Removed |Added

 CC||joc...@herr-schmitt.de




--- Comment #4 from Jochen Schmitt   2009-01-20 
15:01:57 EDT ---
I couldn't download the source rpm. I have got a 404 Page not found.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 458090] Review Request: LuxRender - Lux Renderer, an unbiased rendering system

2009-01-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=458090





--- Comment #5 from Nicolas Chauvet (kwizart)   2009-01-20 
15:15:01 EDT ---
fixed, (deleted the new src.rpm instead of the older by mistake).

@Jochen
btw. I have a very bad problem with the way blender create the ~/.blender
directory nowadays. the main issue is that directories created with the
"blender-wrapper"  script are set root:root even in /home/user directories.
Can we be more in shape with what upstream does in this area ?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 458090] Review Request: LuxRender - Lux Renderer, an unbiased rendering system

2009-01-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=458090





--- Comment #6 from Jochen Schmitt   2009-01-20 
15:34:44 EDT ---
I have a look on my own system. On my system the .blender directory is owned by
my onw userid. The Directory should be create on the first start of blender.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 458090] Review Request: LuxRender - Lux Renderer, an unbiased rendering system

2009-01-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=458090





--- Comment #7 from Nicolas Chauvet (kwizart)   2009-01-20 
15:55:30 EDT ---

LuxBlend v0.5

WARNING: LuxPath "" is not valid

Traceback (most recent call last):
  File "", line 1, in 
  File "/usr/share/blender/scripts/bpymodules/BPyRegistry.py", line 267, in
SaveConfigData
raise Warning(e) # Resend exception as warning
Warning: [Errno 13] Permission denied:
'/home/kwizart/.blender/scripts/bpydata/config/luxblend.cfg'


And as ~/.blender/scripts/bpydata/readme.txt say.the bpydata directory is a
place for plugin to put their data into. Unfortunately, in the current state of
the blender-wrapper script, it is a symlink to
/usr/share/blender/scripts/bpydata, hence, cannot be written unless root.

I don't know if there is a solution that can avoid to duplicate the system
plugins while allowing for system plugins to write their data into by-users
directory.
Can we just patch the config directory for python plugin instead ? (so both
system/users and arch/noarches plugins can use the same config directory?).

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 458090] Review Request: LuxRender - Lux Renderer, an unbiased rendering system

2009-01-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=458090





--- Comment #8 from Nicolas Chauvet (kwizart)   2009-01-20 
16:52:52 EDT ---
This is not easy: 
- to ease the installation of the LuxBlend plugin, I should put a luxrender.cfg
into /usr/share/blender/scripts/bpydata/config/luxblend.cfg and that file need
to be transferred to the user directory even if the ~/.blender directory was
already created but not if the luxrender.cfg file is already present.

Will you accept extensions to your blender-wrapper script ?
Something like:
for f in /etc/blender-wrapper.d ; do 
  sh ${f}
done

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 458090] Review Request: LuxRender - Lux Renderer, an unbiased rendering system

2009-01-21 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=458090





--- Comment #9 from Jochen Schmitt   2009-01-21 
12:29:01 EDT ---
I don't unterstand you issue: 

I have done the following:
1.) Build your package locally.
2.) Install it
3.) Start blender, so the wrapper script should create the symlinks
4.) View the ~/.blender/scripts directory

I have got the following expected result:

lrwxrwxrwx 1 s4504kr s4504kr   42 21. Jan 18:24 LuxBlend_0.1.py ->
/usr/share/blender/scripts/LuxBlend_0.1.py
lrwxrwxrwx 1 s4504kr s4504kr   43 21. Jan 18:24 LuxBlend_0.1.pyc ->
/usr/share/blender/scripts/LuxBlend_0.1.pyc
lrwxrwxrwx 1 s4504kr s4504kr   43 21. Jan 18:24 LuxBlend_0.1.pyo ->
/usr/share/blender/scripts/LuxBlend_0.1.pyo

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 458090] Review Request: LuxRender - Lux Renderer, an unbiased rendering system

2009-01-21 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=458090





--- Comment #10 from Nicolas Chauvet (kwizart)   2009-01-21 
12:44:49 EDT ---
that's not what will append for real:
1- As the per user ~/.blender directory is already created no more files will
be copied to the user directory.
2- Nobody wants to write on LuxBlend_0.1.py?. Instead what the plugin expects
is to write in its config directory and in its content directory. (see the
readme.txt).
Please note that the permission you have showed are the perm of the symlinks,
not the perms of the files.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 458090] Review Request: LuxRender - Lux Renderer, an unbiased rendering system

2009-01-21 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=458090





--- Comment #11 from Jochen Schmitt   2009-01-21 
12:53:20 EDT ---
(In reply to comment #10)
> that's not what will append for real:
> 1- As the per user ~/.blender directory is already created no more files will
> be copied to the user directory.

Sorry, On eatch start of blender there will be creating symlink from
~/.blnder/scripts to /usr/share/blender/scripts.

> 2- Nobody wants to write on LuxBlend_0.1.py?. Instead what the plugin expects
> is to write in its config directory and in its content directory. (see the
> readme.txt).

I can't find your readme.txt file

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 458090] Review Request: LuxRender - Lux Renderer, an unbiased rendering system

2008-08-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=458090





--- Comment #1 from Jason Tibbitts <[EMAIL PROTECTED]>  2008-08-16 13:24:37 EDT 
---
This fails to build for me:

error:
syntax error in expression
error:
/builddir/build/SPECS/LuxRender.spec:79: parseExpressionBoolean returns -1

I've starred line 79:

  %cmake \
-CMAKE_VERBOSE_MAKEFILE:BOOL=TRUE \
-DCMAKE_SKIP_RPATH:BOOL=ON \
* %if %{?_lib} == "lib64"
-DLIBDIR=%{_lib} \
  %endif
  ..

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 458090] Review Request: LuxRender - Lux Renderer, an unbiased rendering system

2008-09-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=458090





--- Comment #2 from Nicolas Chauvet (kwizart) <[EMAIL PROTECTED]>  2008-09-03 
10:54:08 EDT ---
This was fixed in yesterday rpm update

Unfortunately re-submit failed. (need to handle newer boost update probably)
http://koji.fedoraproject.org/koji/getfile?taskID=802082&name=build.log

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review