[frameworks-kholidays] [Bug 414369] Please add Lebanon to the list of holidays.

2019-12-05 Thread Hussam Al-Tayeb
https://bugs.kde.org/show_bug.cgi?id=414369

--- Comment #10 from Hussam Al-Tayeb  ---
(In reply to Christophe Giboudeaux from comment #9)
> You can find examples of islamic holidays in a couple files (holiday_bh_ar,
> holiday_dz_ar, holiday_eg_ar, holiday_ma_ar)

This clarifies it a bit
https://cgit.kde.org/kholidays.git/tree/holidays/plan2/holiday_bh_ar
Thank you.

-- 
You are receiving this mail because:
You are watching all bug changes.

[frameworks-kholidays] [Bug 414369] Please add Lebanon to the list of holidays.

2019-12-05 Thread Christophe Giboudeaux
https://bugs.kde.org/show_bug.cgi?id=414369

Christophe Giboudeaux  changed:

   What|Removed |Added

  Latest Commit|https://commits.kde.org/kho |
   |lidays/748d0219f3f34c729f21 |
   |fe670030a3986c774342|
   Version Fixed In|5.65|

-- 
You are receiving this mail because:
You are watching all bug changes.

[frameworks-kholidays] [Bug 414369] Please add Lebanon to the list of holidays.

2019-12-05 Thread Christophe Giboudeaux
https://bugs.kde.org/show_bug.cgi?id=414369

--- Comment #9 from Christophe Giboudeaux  ---
You can find examples of islamic holidays in a couple files (holiday_bh_ar,
holiday_dz_ar, holiday_eg_ar, holiday_ma_ar)

-- 
You are receiving this mail because:
You are watching all bug changes.

[frameworks-kholidays] [Bug 414369] Please add Lebanon to the list of holidays.

2019-12-05 Thread Luigi Toscano
https://bugs.kde.org/show_bug.cgi?id=414369

Luigi Toscano  changed:

   What|Removed |Added

 Ever confirmed|0   |1
 Resolution|FIXED   |---
 Status|RESOLVED|REOPENED

-- 
You are receiving this mail because:
You are watching all bug changes.

[frameworks-kholidays] [Bug 414369] Please add Lebanon to the list of holidays.

2019-12-05 Thread Luigi Toscano
https://bugs.kde.org/show_bug.cgi?id=414369

--- Comment #8 from Luigi Toscano  ---
Git commit df770235396d5e7c1dd98cdfeb02eea38660df14 by Luigi Toscano.
Committed on 05/12/2019 at 14:18.
Pushed by ltoscano into branch 'master'.

Revert "add Lebanon holidays"

As discussed in the bug, the change is not complete
and having it as it is in the released Frameworks may be
more problematic than not having it.
Specifically, all the religious date should use the
proper rule for dynamic calculation. This applies
to both Christian holidays like Western and Eastern
Easter and all Islamic holidays, which use a
different calendar.

The initial proposal is interesting but it should go
through a few cycles of reviews, addressing the
This reverts commit 748d0219f3f34c729f21fe670030a3986c774342.

M  +0-1holidays/holidays.qrc
D  +0-52   holidays/plan2/holiday_lb_ar

https://commits.kde.org/kholidays/df770235396d5e7c1dd98cdfeb02eea38660df14

-- 
You are receiving this mail because:
You are watching all bug changes.

[frameworks-kholidays] [Bug 414369] Please add Lebanon to the list of holidays.

2019-11-30 Thread Hussam Al-Tayeb
https://bugs.kde.org/show_bug.cgi?id=414369

--- Comment #7 from Hussam Al-Tayeb  ---
I don't mind a revert. This was of course a draft :)

-- 
You are receiving this mail because:
You are watching all bug changes.

[frameworks-kholidays] [Bug 414369] Please add Lebanon to the list of holidays.

2019-11-30 Thread Luigi Toscano
https://bugs.kde.org/show_bug.cgi?id=414369

--- Comment #6 from Luigi Toscano  ---
The question about the calendar is more for the KHolidays maintainers. Of
course they are official 

Unfortunately updating is not a solution: the version of Frameworks may be used
in different years, or not updated. You may define a way, but in general any
recurrent day should be implemented with a formula.

That said, other calendars are supported: see for example this file:
https://cgit.kde.org/kholidays.git/tree/holidays/plan2/holiday_ar_es#n75

Can we please revert it? Right now half of the dates are not properly set. I
suggest also the usage of phabricator to submit and discuss the review.

-- 
You are receiving this mail because:
You are watching all bug changes.

[frameworks-kholidays] [Bug 414369] Please add Lebanon to the list of holidays.

2019-11-30 Thread Hussam Al-Tayeb
https://bugs.kde.org/show_bug.cgi?id=414369

--- Comment #5 from Hussam Al-Tayeb  ---
The islamic ones are a different calendar but are official governmental
holidays. One thing I can do is sync them monthly for each KF5 release. I'm not
sure otherwise. Sorry.

-- 
You are receiving this mail because:
You are watching all bug changes.

[frameworks-kholidays] [Bug 414369] Please add Lebanon to the list of holidays.

2019-11-30 Thread Luigi Toscano
https://bugs.kde.org/show_bug.cgi?id=414369

Luigi Toscano  changed:

   What|Removed |Added

 CC||luigi.tosc...@tiscali.it

--- Comment #4 from Luigi Toscano  ---
Thanks Hussam - Allen, please commit using the correct authorship information.

The file is not completely correct though, as a few mobile holidays (everything
related to Easter, Western and Eastern) should be computed, not have a fixed
day. Not sure about the Islamic ones, which use a different calendar (do we
support it?)

I'd suggest a revert and more time for review.

-- 
You are receiving this mail because:
You are watching all bug changes.

[frameworks-kholidays] [Bug 414369] Please add Lebanon to the list of holidays.

2019-11-30 Thread Hussam Al-Tayeb
https://bugs.kde.org/show_bug.cgi?id=414369

--- Comment #3 from Hussam Al-Tayeb  ---
Thank you very much, Allen. I will do my best to keep it maintained and up to
date.

-- 
You are receiving this mail because:
You are watching all bug changes.

[frameworks-kholidays] [Bug 414369] Please add Lebanon to the list of holidays.

2019-11-30 Thread Allen Winter
https://bugs.kde.org/show_bug.cgi?id=414369

Allen Winter  changed:

   What|Removed |Added

  Latest Commit||https://commits.kde.org/kho
   ||lidays/748d0219f3f34c729f21
   ||fe670030a3986c774342
 Resolution|--- |FIXED
   Version Fixed In||5.65
 Status|REPORTED|RESOLVED

--- Comment #2 from Allen Winter  ---
Git commit 748d0219f3f34c729f21fe670030a3986c774342 by Allen Winter.
Committed on 30/11/2019 at 14:50.
Pushed by winterz into branch 'master'.

add Lebanon holidays

thanks for the contribution Hussam
FIXED-IN: 5.65

M  +1-0holidays/holidays.qrc
A  +52   -0holidays/plan2/holiday_lb_ar

https://commits.kde.org/kholidays/748d0219f3f34c729f21fe670030a3986c774342

-- 
You are receiving this mail because:
You are watching all bug changes.

[frameworks-kholidays] [Bug 414369] Please add Lebanon to the list of holidays.

2019-11-22 Thread Hussam Al-Tayeb
https://bugs.kde.org/show_bug.cgi?id=414369

--- Comment #1 from Hussam Al-Tayeb  ---
Created attachment 124065
  --> https://bugs.kde.org/attachment.cgi?id=124065=edit
draft holiday file

I tried to write a holiday file by imitating the eg_ar file but I'm not sure I
did it correctly or how to install it.

-- 
You are receiving this mail because:
You are watching all bug changes.

[frameworks-kholidays] [Bug 414369] Please add Lebanon to the list of holidays.

2019-11-22 Thread Christophe Giboudeaux
https://bugs.kde.org/show_bug.cgi?id=414369

Christophe Giboudeaux  changed:

   What|Removed |Added

   Severity|normal  |wishlist

-- 
You are receiving this mail because:
You are watching all bug changes.

[frameworks-kholidays] [Bug 414369] Please add Lebanon to the list of holidays.

2019-11-21 Thread Christophe Giboudeaux
https://bugs.kde.org/show_bug.cgi?id=414369

Christophe Giboudeaux  changed:

   What|Removed |Added

  Component|kholidays   |general
Version|GIT (master)|5.64.0
   Assignee|jl...@kde.org   |kdepim-b...@kde.org
Product|kdepimlibs  |frameworks-kholidays

-- 
You are receiving this mail because:
You are watching all bug changes.