[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Re: ParaView package



On Sun, May 16, 2010 at 2:29 PM, Daniel Leidert
<daniel.leidert.spam@gmx.net> wrote:
> Am Sonntag, den 16.05.2010, 11:45 +0200 schrieb Mathieu Malaterre:
>> On Wed, May 12, 2010 at 10:41 PM, Daniel Leidert
>> <daniel.leidert.spam@gmx.net> wrote:
>> > Am Montag, den 10.05.2010, 19:23 +0200 schrieb Mathieu Malaterre:
>> >
>> > [..]
>> >> There are still some lintians errors, but I do not understand why they show up:
>> > [..]
>> >> W: paraview: script-not-executable ./usr/lib/paraview/testing/FindString.tcl
>> >> W: paraview: executable-not-elf-or-script
>> >> ./usr/share/pyshared/paraview/vtk/dataset_adapter.py
>> >> W: paraview: executable-not-elf-or-script
>> >> ./usr/share/pyshared/paraview/vtk/__init__.py
>> >> W: paraview: executable-not-elf-or-script
>> >> ./usr/share/pyshared/paraview/servermanager.py
>> >> W: paraview: executable-not-elf-or-script
>> >> ./usr/share/pyshared/paraview/vtk/algorithms.py
>> >> N: 19 tags overridden (19 errors)
>> >
>> > 19 tags overridden?!
>>
>> I tried to simplify the lintian override to the minimum one. But I am
>> not sure why those are here (maybe because paraview use a convienient
>> copy of VTK -for good reasons-).
>>
>> >> If anyone could tell me what is wrong in the debian/rules file:
>> >>
>> >>
>> >> cat debian/rules:
>> >> ...
>> >>       -chmod a-x $(CURDIR)/usr/share/pyshared/paraview/*.py
>> >>       -chmod a-x $(CURDIR)/usr/share/pyshared/paraview/vtk/*.py
>> >
>> > You have to do this after running dh_pycentral. I guess you run it
>> > before. In this case you have to fix the permissions in the directory
>> > the files were copied/installed to. dh_pycentral then moves them to
>> > their final location.
>>
>> I move those lines around, without success :(
>>
>> >>       -chmod a-x $(CURDIR)/usr/lib/paraview/testing/*.tcl
>> >
>> > This is the opposite of what lintian wants you to do. It complains that
>> > this script *misses* executable permissions.
>>
>> Same thing I changed into a + without no luck :(
>
> Well, it's probably very simple. The path looks weird. Shouldn't it be
> $(CURDIR)/debian/paraview/usr/... instead of $(CURDIR)/usr/?
>
> If that doesn't solve your problem, please provide the build log.

bingo !

...
Current time: Tue May 18 12:42:35 CEST 2010
pbuilder-time-stamp: 1274179355
build command was successful; binaries are in
/home/mathieu/debian/debian-science/packages/paraview/build-area/. The
changes file is:
 /home/mathieu/debian/debian-science/packages/paraview/build-area/paraview_3.6.2-4_amd64.changes
Binary package:
 /home/mathieu/debian/debian-science/packages/paraview/build-area/paraview_3.6.2-4_amd64.deb
lintian /home/mathieu/debian/debian-science/packages/paraview/build-area/paraview_3.6.2-4_amd64.changes
N: 19 tags overridden (19 errors)
...

I think paraview is now is pretty darn good shape ! feel free to upload.

Thanks,
-- 
Mathieu


Reply to: