Simon,

Are you now ready for a vote on telepathy-glib then?

Amanda


simon wrote:
> Amanda Waite wrote:
>> Hi Simon,
>>
>> The new package changes the permissions of /usr/include, /usr/share, 
>> /usr/share/doc to 444 and the permissions of /usr and /usr/lib and 
>> /usr/lib/pkgconfig to 555. I'd advise you not to try installing it as 
>> it took me a while to fix my system :o). This is happening mainly 
>> because of the lines:
>>
>> %dir %attr (0555, root, bin) %{_libdir}
>> %dir %attr (0444, root, sys) %{_datadir}
>> %dir %attr (0444, root, other) %{_datadir}/doc
>>
>> Which act recursively on the directory and it's contents.
>>
>> Regards
>>
>> Amanda
>>
>>
>
> It works by using 'find $RPM_BUILD_DIR/usr/include -name \*.h -exec 
> chmod 444 '{}' \; in the %install section.
> Now permissions on all the header files is 444 and permissions on the 
> shared library is 555.
> Thanks.
>
> Simon
>>
>>
>> simon wrote:
>>> Amanda Waite wrote:
>>>> yuntong.jin wrote:
>>>>> Hi Amanda,
>>>>>         Thanks for below comments on telepathy-glib in juicer
>>>>>
>>>>> ""Things that I'd like to see addressed before I would vote this in:
>>>>>
>>>>> add field Meta(info.classification) see 
>>>>> http://opensolaris.org/os/community/sw-porters/contributing/ipsclass/ 
>>>>> for classes. The format would be org.opensolaris.category.2008:/
>>>>>
>>>>> 'Meta(info.repository_url)' should be the URL to the source code 
>>>>> repository (svn, cvs, git, hg, etc). If there is no external 
>>>>> access to the repository then leave this field out.
>>>>>
>>>>> permissions on all the header files should be 444 and permissions 
>>>>> on the shared library should be 555
>>>>>
>>>>> The README file (/usr/share/doc/telepathy-glib/README) mentions a 
>>>>> spec/ directory but I don't see one.
>>>>>
>>>>> add an entry to the changelog in the spec file to say when you 
>>>>> made the last change to the spec""
>>>>>
>>>>>
>>>>> I have some questions about it:
>>>>>
>>>>> Should  permissions on all the header files the shared library be 
>>>>> 755? Those files are set 0755 in almost all specs .
>>>>> Files under spec/ is not installed in Makefile, should i include 
>>>>> them in package?
>>>> It's a hangover from my SFW involvement where the recommendation is 
>>>> (supposedly) that files installed in the top level /usr directories 
>>>> shouldn't have the write bit set. To be honest I'll be happy if the 
>>>> library is 755 and the headers are 644, currently even the header 
>>>> files are 755 and that isn't right. As for the spec/ directory, I 
>>>> only mention it because the README does, only you can say if it's 
>>>> needed or not.
>>>>
>>>> Amanda
>>> I have updated according to your comments, pls review it .
>>> http://jucr.opensolaris.org/review/packages/641/
>>>
>>> Latest successful build in here.
>>> http://jucr.opensolaris.org/build/viewjob/3537/
>>>
>>> BR,
>>> -Simon
>>>
>>>>>
>>>>> http://jucr.opensolaris.org/review/packages/641/
>>>>>
>>>>> BR,
>>>>> -Simon
>>>>>
>>>>>
>>>>>
>>>>>
>>>>>
>>>>>
>>>>>
>>>>
>>>
>>
>

Reply via email to