{"action":"created","issue":{"url":"https://api.github.com/repos/FreifunkLippe/site-ffl/issues/18","repository_url":"https://api.github.com/repos/FreifunkLippe/site-ffl","labels_url":"https://api.github.com/repos/FreifunkLippe/site-ffl/issues/18/labels{/name}","comments_url":"https://api.github.com/repos/FreifunkLippe/site-ffl/issues/18/comments","events_url":"https://api.github.com/repos/FreifunkLippe/site-ffl/issues/18/events","html_url":"https://github.com/FreifunkLippe/site-ffl/pull/18","id":149959435,"number":18,"title":"V2016.1.x experimental","user":{"login":"collimas","id":11668783,"avatar_url":"https://avatars.githubusercontent.com/u/11668783?v=3","gravatar_id":"","url":"https://api.github.com/users/collimas","html_url":"https://github.com/collimas","followers_url":"https://api.github.com/users/collimas/followers","following_url":"https://api.github.com/users/collimas/following{/other_user}","gists_url":"https://api.github.com/users/collimas/gists{/gist_id}","starred_url":"https://api.github.com/users/collimas/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/collimas/subscriptions","organizations_url":"https://api.github.com/users/collimas/orgs","repos_url":"https://api.github.com/users/collimas/repos","events_url":"https://api.github.com/users/collimas/events{/privacy}","received_events_url":"https://api.github.com/users/collimas/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":0,"created_at":"2016-04-21T04:06:52Z","updated_at":"2016-04-21T05:42:17Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/FreifunkLippe/site-ffl/pulls/18","html_url":"https://github.com/FreifunkLippe/site-ffl/pull/18","diff_url":"https://github.com/FreifunkLippe/site-ffl/pull/18.diff","patch_url":"https://github.com/FreifunkLippe/site-ffl/pull/18.patch"},"body":"Modified build script for faster building and creating log files"},"comment":{"url":"https://api.github.com/repos/FreifunkLippe/site-ffl/issues/comments/212752486","html_url":"https://github.com/FreifunkLippe/site-ffl/pull/18#issuecomment-212752486","issue_url":"https://api.github.com/repos/FreifunkLippe/site-ffl/issues/18","id":212752486,"user":{"login":"Tronde","id":3941069,"avatar_url":"https://avatars.githubusercontent.com/u/3941069?v=3","gravatar_id":"","url":"https://api.github.com/users/Tronde","html_url":"https://github.com/Tronde","followers_url":"https://api.github.com/users/Tronde/followers","following_url":"https://api.github.com/users/Tronde/following{/other_user}","gists_url":"https://api.github.com/users/Tronde/gists{/gist_id}","starred_url":"https://api.github.com/users/Tronde/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Tronde/subscriptions","organizations_url":"https://api.github.com/users/Tronde/orgs","repos_url":"https://api.github.com/users/Tronde/repos","events_url":"https://api.github.com/users/Tronde/events{/privacy}","received_events_url":"https://api.github.com/users/Tronde/received_events","type":"User","site_admin":false},"created_at":"2016-04-21T05:42:17Z","updated_at":"2016-04-21T05:42:17Z","body":"Hallo,\r\n\r\nmir sind im Skript  build_fflip_fw.sh ein paar Punkte aufgefallen, die du noch verbessern könntest.\r\n\r\n 1. Im ersten IF-Zweig prüfst du, ob `$DIR/gluon` existiert und kopierst bei positiver Prüfung Dateien nach `$DIR/gluon/site/`. Ob letzteres Verzeichnis überhaupt existiert prüfst du jedoch erst zwei IF-Blöcke weiter unten.\r\n \r\n Entweder zu ziehst diese Prüfung nach oben und baust eine verschachtelte IF-Anweisung oder lässt diese Prüfung entfallen. Denn entweder kannst du implizit annehmen, dass das Verzeichnis existiert, bevor du Daten hinein kopierst, oder musst diesen Test vorher machen.\r\n\r\n 2. Du kannst die beiden ersten IF-Anweisungen durch `else` verknüpfen. Dies optimiert den Programmablauf.\r\n\r\n 3. Ich sehe zwei For-Schleifen. Hast du mal geprüft, ob sich diese vielleicht in einer zusammenfassen lassen? Falls nein, woran hakt es? Vielleicht habe ich noch eine Idee.\r\n\r\nDann habe ich noch ein paar Fragen. Wofür brauchst du die Commit-ID in der modules? Sollen die Site-Dateien tatsächlich alle mit ins Repository oder war dies nur ein versehen? Diese werden doch bei jedem Build neu erstellt, oder nicht?\r\n\r\nLG\r\nTronde"}}