TDE
/
tdeartwork
6
0
Fork 0
Κώδικας Ζητήματα 1 Pull Requests Κυκλοφορίες Wiki Δραστηριότητα
Σήματα Ορόσημα

Add tdeasciiquarium screen saver #16

Συγχωνευμένα
MicheleC συγχώνευσε 6 υποβολές από feature/bug_1434 σε master 3 έτη πριν
Συζήτηση 9 Υποβολές 6 Αρχεία Άλλαξαν 20
Ghost σχολίασε 3 έτη πριν

Upon completion, bug 1434 from Bugzilla may be closed.

Upon completion, bug 1434 from Bugzilla may be closed.
Ghost πρόσθεσε τη σήμανση PR/wip 3 έτη πριν
SlavekB σχολίασε 3 έτη πριν
Ιδιοκτήτης

Place it in the root directory is not a good choice. Because this is TDE screen saver, it should be in the tdescreensaver/kdesavers directory.

Place it in the root directory is not a good choice. Because this is TDE screen saver, it should be in the `tdescreensaver/kdesavers` directory.
MicheleC σχολίασε 3 έτη πριν
Ιδιοκτήτης

Nice!! Looking forward for this one!

Nice!! Looking forward for this one!
Ghost force-pushed feature/bug_1434 από το 9b3e04a294 στο a6dd5f7080 3 έτη πριν
Ghost σχολίασε 3 έτη πριν
Συντάκτης

@SlavekB

For consistency with the other modules:

  • the option BUILD_ALL have been passed to "ON" by default
  • added symbols visibility option
  • passed WITH_ALL_INCLUDED_XSCREENSAVERS and WITH_ARTS options to the value of ${WITH_ALL_OPTIONS}
@SlavekB For consistency with the other modules: - the option BUILD_ALL have been passed to "ON" by default - added symbols visibility option - passed WITH_ALL_INCLUDED_XSCREENSAVERS and WITH_ARTS options to the value of ${WITH_ALL_OPTIONS}
Ghost αφαίρεσε το σήμα PR/wip 3 έτη πριν
Ghost άλλαξε το τίτλο από WIP:Add asciiquarium screen saver σε Add tdeasciiquarium screen saver 3 έτη πριν
MicheleC σχολίασε 3 έτη πριν
Ιδιοκτήτης

@Greg: just to be sure, this is ready for testing and merging?

@Greg: just to be sure, this is ready for testing and merging?
Ghost σχολίασε 3 έτη πριν
Συντάκτης

Yes, please test.

Yes, please test.
Ghost force-pushed feature/bug_1434 από το a6dd5f7080 στο 18732d51a3 3 έτη πριν
SlavekB σχολίασε 3 έτη πριν
Ιδιοκτήτης

Please can you use for example a prefix "asciiquarium:" in git commit comments? Existing comments without specifying an application seem confusing.

Please can you use for example a prefix "asciiquarium:" in git commit comments? Existing comments without specifying an application seem confusing.
MicheleC σχολίασε 3 έτη πριν
Ιδιοκτήτης

Tested in bullseye, looks beautiful 👍

Tested in bullseye, looks beautiful :+1:
Ghost force-pushed feature/bug_1434 από το 18732d51a3 στο 13c31402df 3 έτη πριν
SlavekB ενέκρινε αυτές τις αλλαγές 3 έτη πριν
SlavekB άφησε ένα σχόλιο
Ιδιοκτήτης

Thank you Greg, it looks great.

Thank you Greg, it looks great.
MicheleC σχολίασε 3 έτη πριν
Ιδιοκτήτης

Before we merge this, I think it would be good to add in the message of the first commit (cfee243b43) where we took the code from (http://kde-look.org/content/show.php?content=29207 and that the original asciiquarium code is available under GPL@ license (http://www.robobunny.com/projects/asciiquarium, file gpl.txt). Just for the sake of clarity.

Before we merge this, I think it would be good to add in the message of the first commit (cfee243b43) where we took the code from (http://kde-look.org/content/show.php?content=29207 and that the original asciiquarium code is available under GPL@ license (http://www.robobunny.com/projects/asciiquarium, file gpl.txt). Just for the sake of clarity.
Ghost force-pushed feature/bug_1434 από το 13c31402df στο d9d663485e 3 έτη πριν
MicheleC συγχώνευσε την υποβολή d9d663485e σε master 3 έτη πριν
MicheleC διέγραψε το κλάδο feature/bug_1434 3 έτη πριν
MicheleC σχολίασε 3 έτη πριν
Ιδιοκτήτης

Thanks Greg, excellent work!

Thanks Greg, excellent work!
MicheleC το πρόσθεσε στο R14.0.11 release ορόσημο 3 έτη πριν

Εξεταστές

SlavekB ενέκρινε αυτές τις αλλαγές 3 έτη πριν
Το pull request έχει συγχωνευθεί ως d9d663485e.
Συνδεθείτε για να συμμετάσχετε σε αυτή τη συνομιλία.
Εξεταστές
Αίτηση επανεξέτασης
Δεν υπάρχουν εξεταστές
SlavekB
Σήματα
Εφαρμογή σημάτων
Καθαρισμός σημάτων
GE/need-info
General - need additional info from contributor PR/keep-branch
Pull request - do not delete branch after merging PR/not-ok
Pull request - need fixing PR/rfc
Pull request - request for comments PR/update-trans
Pull request - update to translation files needed PR/wip
Pull request - work in progress RS/R14.0.x
Related to R14.0.x series RS/R14.1.x
Related to R14.1.x series SL/critical
Severity level - critical SL/major
Severity level - major SL/minor
Severity level - minor SL/normal
Severity level - normal SL/regression
Severity level - regression from previous version SL/trivial
Severity level - trivial SL/wishlist
Severity level - wishlist request ST/duplicate
Status - duplicate of another issue ST/invalid
Status - invalid report ST/notourproblem
Status - not our problem ST/rejected
Status - rejected ST/wontfix
Status - won't fix ST/worksforme
Status - works for me, unable to reproduce
Χωρίς Σήμα GE/need-info PR/keep-branch PR/not-ok PR/rfc PR/update-trans PR/wip RS/R14.0.x RS/R14.1.x SL/critical SL/major SL/minor SL/normal SL/regression SL/trivial SL/wishlist ST/duplicate ST/invalid ST/notourproblem ST/rejected ST/wontfix ST/worksforme
Ορόσημο
Ορισμός ορόσημου
Καθαρισμός ορόσημου
Δεν υπάρχουν αντικείμενα
Χωρίς Ορόσημο
R14.0.11 release
Αποδέκτες
Ανάθεση χρηστών
Εκκαθάριση αποδεκτών
Χωρίς Αποδέκτη
3 Συμμετέχοντες
Ειδοποιήσεις
Ημερομηνία Παράδοσης

Δεν ορίστηκε ημερομηνία παράδοσης.

Εξαρτήσεις

Δεν έχουν οριστεί εξαρτήσεις.

Αναφορά: TDE/tdeartwork#16
Σύνταξη Προεπισκόπηση
Φόρτωση…
Ακύρωση
Αποθήκευση
Δεν υπάρχει ακόμα περιεχόμενο.
Διαγραφή του Κλάδου 'feature/bug_1434'

Η διαγραφή του κλάδου είναι μόνιμη. ΔΕΝ ΜΠΟΡΕΙ να αναιρεθεί. Συνέχεια;

Όχι
Ναι