Overview

Request 123709 superseded

update OTRS to 3.1.6
update OTRS:ITSM to 3.1.4 (forwarded request 123708 from computersalat)

Loading...
Request History
Christian Wittmer's avatar

computersalat created request

update OTRS to 3.1.6
update OTRS:ITSM to 3.1.4 (forwarded request 123708 from computersalat)


Factory Auto's avatar

factory-auto added a reviewer

please make sure to wait before these depencencies are in openSUSE:Factory: otrs:perl(PDF::API2)


Stephan Kulow's avatar

coolo declined request

otrs requires perl(PDF::API2), which is no longer in factory (and can't be updated as that perl module requires a non-free module ;(

So either this requires can be made optional, or we can't keep otrs in factory


Christian Wittmer's avatar

computersalat added a reviewer

I see no problem with perl(PDF::API2) dependency perl(Font::TTF) which license is Artistic-2.0
see SR 124256 for perl-Font-TTF, and SR 124257 for perl-PDF-API2


Factory Auto's avatar

factory-auto added a reviewer

please make sure to wait before these depencencies are in openSUSE:Factory: otrs:perl(PDF::API2)


Factory Auto's avatar

factory-auto declined request

Output of check script:
Source validator failed. Try "osc service localrun source_validator"
(E) Attention, otrs.permissions is not mentioned in spec files as source or patch.
- package has otrs.rpmlintrc: (new or modified)
--------------------------------------------------------------
addFilter('files-duplicate /srv/otrs/Kernel/Config.pm.dist')
addFilter('files-duplicate /srv/otrs/Kernel/Config/GenericAgent.pm.dist')
addFilter('hidden-file-or-dir')
addFilter('macro-in-comment')
addFilter('non-etc-or-var-file-marked-as-conffile')
addFilter('non-executable-script')
addFilter('zero-length')
addFilter('permissions-unauthorized-file')
--------------------------------------------------------------
###ASK otrs/otrs.rpmlintrc
WARNING: macro used in version line
packager is responsible that the resulting string
is non-empty and contains no "-".
----------
Name: otrs
Version: %{otrs_ver}
%package -n otrs-itsm
Version: %{itsm_ver}
----------
--------------------------------------------------------------
otrs.spec has a %pre with groupadd and/or useradd
please check if this is ok:

-----------------master-table-entry:--------------------------
/work/cd/lib/misc/group:otrs otrs:x:YY:
/work/cd/lib/misc/passwd:otrs otrs:x:XX:YY:OTRS user:/opt/otrs:/bin/false

-----------------specfile section:----------------------------
%preun
%stop_on_removal otrs
%pre
# on `rpm -ivh` PARAM is 1
# on `rpm -Uvh` PARAM is 2
if [ -z "`%{_bindir}/getent group %{name} 2>/dev/null`" ]; then
%{_sbindir}/groupadd -g 88 -r %{name} 2>/dev/null
else
if [ `%{_bindir}/id -g %{name}` != 88 ]; then
%{_sbindir}/groupmod -g 88 %{name} 2>/dev/null
fi
fi
if [ -z "`%{_bindir}/getent passwd %{name} 2>/dev/null`" ]; then
%{_sbindir}/useradd -c "OTRS User" -d %{otrs_root}/%{name} -G %{name},www -g %{name} -u 88 -r -s /bin/false %{name} 2>/dev/null
else
if [ `%{_bindir}/id -u %{name}` != 88 ]; then
%{_sbindir}/usermod -c "OTRS User" -d %{otrs_root}/%{name} -G %{name},www -g %{name} -u 88 %{name} 2>/dev/null
fi
fi

%post
--------------------------------------------------------------


Christian Wittmer's avatar

computersalat superseded request

superseded by 124419


Saul Goodman's avatar

licensedigger accepted review

{"approve": "preliminary, version number changed"}


Factory Auto's avatar

factory-auto declined review

Output of check script:
Source validator failed. Try "osc service localrun source_validator"
(E) Attention, otrs.permissions is not mentioned in spec files as source or patch.
- package has otrs.rpmlintrc: (new or modified)
--------------------------------------------------------------
addFilter('files-duplicate /srv/otrs/Kernel/Config.pm.dist')
addFilter('files-duplicate /srv/otrs/Kernel/Config/GenericAgent.pm.dist')
addFilter('hidden-file-or-dir')
addFilter('macro-in-comment')
addFilter('non-etc-or-var-file-marked-as-conffile')
addFilter('non-executable-script')
addFilter('zero-length')
addFilter('permissions-unauthorized-file')
--------------------------------------------------------------
###ASK otrs/otrs.rpmlintrc
WARNING: macro used in version line
packager is responsible that the resulting string
is non-empty and contains no "-".
----------
Name: otrs
Version: %{otrs_ver}
%package -n otrs-itsm
Version: %{itsm_ver}
----------
--------------------------------------------------------------
otrs.spec has a %pre with groupadd and/or useradd
please check if this is ok:

-----------------master-table-entry:--------------------------
/work/cd/lib/misc/group:otrs otrs:x:YY:
/work/cd/lib/misc/passwd:otrs otrs:x:XX:YY:OTRS user:/opt/otrs:/bin/false

-----------------specfile section:----------------------------
%preun
%stop_on_removal otrs
%pre
# on `rpm -ivh` PARAM is 1
# on `rpm -Uvh` PARAM is 2
if [ -z "`%{_bindir}/getent group %{name} 2>/dev/null`" ]; then
%{_sbindir}/groupadd -g 88 -r %{name} 2>/dev/null
else
if [ `%{_bindir}/id -g %{name}` != 88 ]; then
%{_sbindir}/groupmod -g 88 %{name} 2>/dev/null
fi
fi
if [ -z "`%{_bindir}/getent passwd %{name} 2>/dev/null`" ]; then
%{_sbindir}/useradd -c "OTRS User" -d %{otrs_root}/%{name} -G %{name},www -g %{name} -u 88 -r -s /bin/false %{name} 2>/dev/null
else
if [ `%{_bindir}/id -u %{name}` != 88 ]; then
%{_sbindir}/usermod -c "OTRS User" -d %{otrs_root}/%{name} -G %{name},www -g %{name} -u 88 %{name} 2>/dev/null
fi
fi

%post
--------------------------------------------------------------

openSUSE Build Service is sponsored by