311246 - remove other-licenses/libical

Status: RESOLVED FIXED

(Calendar :: Internal Components, defect)

Description

[Michiel van Leeuwen (email: mvl+moz@)]

other-licenses/libical isn't used anymore. We should remove it.

Comment 1

[Michiel van Leeuwen (email: mvl+moz@)]

Attachment: patch

This patch simply replaces other-licenses/libical with calendar/libical in
allmakefiles.sh and modules.mk.
Also part of this checkin would be the actual removal of
other-licenses/libical, but i didn't include that in the patch for obvious
reasons.

Comment 2

[Dan Mosedale (:dmosedale, :dmose)]

Comment on attachment 198613 [details] [diff] [review]
patch

r=dmose

Comment 3

[Michiel van Leeuwen (email: mvl+moz@)]

Comment on attachment 198613 [details] [diff] [review]
patch

I almost forgot that those buildconfig changes don't fall under calendar rules.

Comment 4

[Benjamin Smedberg]

Comment on attachment 198613 [details] [diff] [review]
patch

yah, yah

Comment 5

[Michiel van Leeuwen (email: mvl+moz@)]

patch checked in some time ago. Bonsai confirms it. lxr disagrees :( Still need
to figure that out.

Comment 6

[Robin Edrenius]

(In reply to comment #5)
> patch checked in some time ago. Bonsai confirms it. lxr disagrees :( Still need
> to figure that out.

It's checked in according to lxr too now. 
Should this be marked as fixed or do other-licenses/libical have to be removed first (comment #1)?

Comment 7

[Michiel van Leeuwen (email: mvl+moz@)]

other-licenses/libical is cvs removed, but libical still shows it. That's what i was talking about.

Comment 8

[Michiel van Leeuwen (email: mvl+moz@)]

this has been fixed for quite some time now.

Comment 9

[Reed Loden [:reed]]

The bugspam monkeys have been set free and are feeding on Calendar :: Internal Components. Be afraid for your sanity!