ClubCedille / jardiniot

Système pour des buckets autonomes servant à faire pousser des plantes
GNU General Public License v3.0
13 stars 11 forks source link

Sync à mon repo personnel #87

Closed Perceval62 closed 4 years ago

Perceval62 commented 4 years ago

Cet pull request a pour but de synchroniser le travail fait dans mon repo personnel à ma branche dédiée du projet sur ce répo.

AXDOOMER commented 4 years ago

Copyright headers à utiliser pour le code qui utilise du code de JardinIoT.

// Copyright (C) 2016-2018 Alexandre-Xavier Labonté-Lamoureux
// Copyright (C) 2017      Alexandre Brochu
//
// JardinIoT is free software: you can redistribute it and/or modify
// it under the terms of the GNU General Public License as published by
// the Free Software Foundation, either version 3 of the License, or
// (at your option) any later version.
//
// JardinIoT is distributed in the hope that it will be useful,
// but WITHOUT ANY WARRANTY; without even the implied warranty of
// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
// GNU General Public License for more details.
//
// You should have received a copy of the GNU General Public License
// along with JardinIoT.  If not, see <http://www.gnu.org/licenses/>.
Perceval62 commented 4 years ago

J'ai passé le tout à clang-format, créer un script pour rouler clang-format sur tous les fichiers importants. Je suis en train d'ajouter un Vagrantfile

mikefaille commented 4 years ago

@Perceval62 Vagrant serait peut-être trop long à rouler. C'est un excellent cas d'utilisation pour docker.

Perceval62 commented 4 years ago

Je vais enlever le fichier pour le moment, je veux seulement merge pour centraliser le développement.

mikefaille commented 4 years ago

Le merge est pas mal gros en effet ! ;-)

Perceval62 commented 4 years ago

Ouais, j'attends juste après @AXDOOMER et je vais arrêter de gosser avec de PR ben trop grosses lol

mikefaille commented 4 years ago

C'est une PR ou un fork ?:p

Perceval62 commented 4 years ago

J'ai forké au début pcq je n'avais pas les accès... Une fois cette PR fermé le développement se fera seulement dans ce repo ci

Perceval62 commented 4 years ago

Ok, J'ai assez travaillé la dessus aujourd'hui. J'attends l'approbation de @AXDOOMER

mikefaille commented 4 years ago

Je suggère de crée un ticket pour ça. On pourrait se permettre de faire plusieurs petites maintenance du genre en même temps par plusieurs personnes après.

Le mer. 13 mai 2020 15:37, Nicolas Berbiche notifications@github.com a écrit :

@berbiche commented on this pull request.

In jardiniot-emb/rev2/main/src/initialization/system_init.c https://github.com/ClubCedille/jardiniot/pull/87#discussion_r424684310:

  • address.sin_addr.s_addr = INADDR_ANY;
  • address.sin_port = htons( PORT );

  • bind(sockfd, (struct sockaddr *)&address, sizeof(address));

  • if (listen(sockfd, 3) < 0)

  • {

  • perror("listen");

  • exit(EXIT_FAILURE);

  • }

  • if ((new_socket = accept(sockfd, (struct sockaddr *)&address,

  • (socklen_t*)&addrlen))<0)

  • {

  • perror("accept");

  • exit(EXIT_FAILURE);

  • }

  • valread = read( new_socket , buffer, 1024);

  • printf(buffer);

Ce qui me fait pensé, as-tu activé les flags nécessaires pour que le compilateur report l'utilisation direct d'une variable avec printf sans passer de string de formattage? Genre: printf(variable) plutôt que printf("%s", variable)

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/ClubCedille/jardiniot/pull/87#discussion_r424684310, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAHO2FBKLSFZZQDFF3CFSG3RRLZHBANCNFSM4M7FEJDQ .